[DFG][FTL] Support Array::DirectArguments with OutOfBounds
Created attachment 326713 [details] Patch WIP
Created attachment 326714 [details] Patch WIP
Created attachment 326720 [details] Patch
Created attachment 326876 [details] Patch
Comment on attachment 326876 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=326876&action=review > Source/JavaScriptCore/dfg/DFGArgumentsEliminationPhase.cpp:245 > + m_graph.registerAndWatchStructureTransition(objectPrototypeStructure); Don’t you want to check OOB mode here? Or is the !prototypeChaneIsSane rare enough that we just do it unconditionally?
Comment on attachment 326876 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=326876&action=review Thank you! >> Source/JavaScriptCore/dfg/DFGArgumentsEliminationPhase.cpp:245 >> + m_graph.registerAndWatchStructureTransition(objectPrototypeStructure); > > Don’t you want to check OOB mode here? Or is the !prototypeChaneIsSane rare enough that we just do it unconditionally? Oh, nice catch. We should insert if (mode.isInBounds()) break; check before this to align it to ClonedArguments case.
Committed r224818: <https://trac.webkit.org/changeset/224818>
<rdar://problem/35562030>