The DFG JIT's support for comparisons and branching may emit tests to see if the operands are integers. If they are, the comparison can be done more quickly by doing an Int32 compare. But in some cases, the compiler knows statically that one or both of the operands are definitely not integers. In that case, the JIT should always emit a slow call and omit the unnecessary is-int test.
Created attachment 101098 [details] the patch
Created attachment 101099 [details] the patch (fix Platform.h) Previous patch accidentally had a change to Platform.h that it should not have had.
Attachment 101098 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/ChangeLog', u'Source..." exit_code: 1 Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:437: One line control clauses should not use braces. [whitespace/braces] [4] Total errors found: 1 in 5 files If any of these errors are false positives, please file a bug against check-webkit-style.
Attachment 101099 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/ChangeLog', u'Source..." exit_code: 1 Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:437: One line control clauses should not use braces. [whitespace/braces] [4] Total errors found: 1 in 4 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 101100 [details] the patch (fix style)
Comment on attachment 101100 [details] the patch (fix style) Clearing flags on attachment: 101100 Committed r91160: <http://trac.webkit.org/changeset/91160>
All reviewed patches have been landed. Closing bug.