Bug 130300

Summary: Add CompareStrictEq(StringIdent:, NotStringVar:) and CompareStrictEq(String:, Untyped:)
Product: WebKit Reporter: Filip Pizlo <fpizlo>
Component: JavaScriptCoreAssignee: Filip Pizlo <fpizlo>
Status: RESOLVED FIXED    
Severity: Normal CC: barraclough, ggaren, kling, mark.lam, mhahnenberg, mmirman, msaboff, nrotem, oliver, ossy, sam
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 130449    
Bug Blocks: 112840    
Attachments:
Description Flags
work in progress
none
moar!
none
getting there
none
basically done
none
the patch mhahnenberg: review+

Description Filip Pizlo 2014-03-15 23:14:07 PDT
Patch forthcoming.
Comment 1 Filip Pizlo 2014-03-16 17:57:30 PDT
Created attachment 226867 [details]
work in progress
Comment 2 Filip Pizlo 2014-03-16 18:24:55 PDT
Created attachment 226868 [details]
moar!
Comment 3 Filip Pizlo 2014-03-16 18:27:08 PDT
I'm using this as an opportunity to add is-cell/is-not-cell/is-other/is-not-other branch abstractions to deduplicate more 64-bit/32-bit code.
Comment 4 Filip Pizlo 2014-03-16 21:28:47 PDT
Created attachment 226876 [details]
getting there

I wrote all of the DFG code, but there is still some FTL code to write.
Comment 5 Filip Pizlo 2014-03-17 21:53:55 PDT
Created attachment 227011 [details]
basically done
Comment 6 Filip Pizlo 2014-03-18 12:35:14 PDT
Created attachment 227089 [details]
the patch
Comment 7 Mark Hahnenberg 2014-03-18 13:37:35 PDT
Comment on attachment 227089 [details]
the patch

View in context: https://bugs.webkit.org/attachment.cgi?id=227089&action=review

r=me

> Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:3859
> +    JITCompiler::JumpList fastSlow;

fastFalse
Comment 8 Filip Pizlo 2014-03-18 13:54:17 PDT
Landed in http://trac.webkit.org/changeset/165842
Comment 9 Csaba Osztrogonác 2014-03-19 04:14:03 PDT
(In reply to comment #8)
> Landed in http://trac.webkit.org/changeset/165842

It caused regression on 32 bit x86 - https://bugs.webkit.org/show_bug.cgi?id=130449