RESOLVED FIXED 117262
Disambiguate between CallFrame bytecodeOffset and codeOriginIndex 
https://bugs.webkit.org/show_bug.cgi?id=117262
Summary Disambiguate between CallFrame bytecodeOffset and codeOriginIndex
Mark Lam
Reported 2013-06-05 10:12:26 PDT
We should be able to discern whether a CallFrame (once fully reified) ArgumentCount tag is a bytecodeOffset or a codeOriginIndex. This change will flag the codeOriginIndex by encoding it as a handle which always has the high bit set. If the ArgumentCount tag word has its high bit set, then we have a codeOriginIndex, else we have a bytecodeOffset.
Attachments
the patch. (9.59 KB, patch)
2013-06-05 18:27 PDT, Mark Lam 		no flags
DetailsFormatted DiffDiff
corrected patch. (25.61 KB, patch)
2013-06-05 18:43 PDT, Mark Lam 		ggaren: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Mark Lam
Comment 1 2013-06-05 18:27:14 PDT
Created attachment 203896 [details] the patch.
Mark Lam
Comment 2 2013-06-05 18:27:54 PDT
Comment on attachment 203896 [details] the patch. Forgot the ChangeLogs. Will re-upload with ChangeLogs.
Mark Lam
Comment 3 2013-06-05 18:43:42 PDT
Created attachment 203897 [details] corrected patch.
Geoffrey Garen
Comment 4 2013-06-05 18:56:00 PDT
Comment on attachment 203897 [details] corrected patch. View in context: https://bugs.webkit.org/attachment.cgi?id=203897&action=review r=me > Source/JavaScriptCore/interpreter/CallFrame.cpp:45 > +unsigned CallFrame::locationAsBytecodeOffset() const This should ASSERT hasLocationAsBytecodeOffset(). > Source/JavaScriptCore/interpreter/CallFrame.cpp:51 > +void CallFrame::setLocationAsBytecodeOffset(unsigned offset) This should ASSERT hasLocationAsBytecodeOffset() after the set -- or ASSERT that offset is not a CodeOrigin handle. > Source/JavaScriptCore/interpreter/CallFrameInlines.h:62 > +inline void CallFrame::setLocationAsBytecodeOffset(unsigned offset) This should ASSERT hasLocationAsBytecodeOffset() after the set -- or ASSERT that offset is not a CodeOrigin handle.
Mark Lam
Comment 5 2013-06-05 19:14:03 PDT
Landed in r151250: <http://trac.webkit.org/changeset/151250>.
Note You need to log in before you can comment on or make changes to this bug.