RESOLVED FIXED 157213
ToThis should be able to be eliminated in Constant Folding 
https://bugs.webkit.org/show_bug.cgi?id=157213
Summary ToThis should be able to be eliminated in Constant Folding
Keith Miller
Reported 2016-04-29 16:16:37 PDT
ToThis should be able to be eliminated in Constant Folding
Attachments
Patch (7.45 KB, patch)
2016-04-29 20:44 PDT, Keith Miller 		no flags
DetailsFormatted DiffDiff
Patch for landing (7.42 KB, patch)
2016-05-02 09:38 PDT, Keith Miller 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Keith Miller
Comment 1 2016-04-29 20:44:51 PDT
Created attachment 277790 [details] Patch
Saam Barati
Comment 2 2016-05-01 16:11:25 PDT
Comment on attachment 277790 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=277790&action=review r=me > Source/JavaScriptCore/dfg/DFGAbstractInterpreterInlines.h:146 > + ASSERT(!isStrictMode ? (type.type() != StringType && type.type() != SymbolType) : true); I think this is easier to read as: if (!strictMode) ASSERT(type.type() != StringType && type.type() != SymbolType)
Keith Miller
Comment 3 2016-05-02 09:38:44 PDT
Created attachment 277910 [details] Patch for landing
Keith Miller
Comment 4 2016-05-02 09:40:03 PDT
Comment on attachment 277790 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=277790&action=review >> Source/JavaScriptCore/dfg/DFGAbstractInterpreterInlines.h:146 >> + ASSERT(!isStrictMode ? (type.type() != StringType && type.type() != SymbolType) : true); > > I think this is easier to read as: > if (!strictMode) > ASSERT(type.type() != StringType && type.type() != SymbolType) I agree, although I actually changed it to: if (!isStrictMode) ASSERT(type.isObjet());
WebKit Commit Bot
Comment 5 2016-05-02 10:37:42 PDT
Comment on attachment 277910 [details] Patch for landing Clearing flags on attachment: 277910 Committed r200325: <http://trac.webkit.org/changeset/200325>
WebKit Commit Bot
Comment 6 2016-05-02 10:37:45 PDT
All reviewed patches have been landed. Closing bug.
Geoffrey Garen
Comment 7 2016-05-02 11:20:06 PDT
Comment on attachment 277910 [details] Patch for landing Seems like we want a performance regression test for this improvement too.
Note You need to log in before you can comment on or make changes to this bug.