Bug 68140 - DFG JIT does not leverage integer speculations on branches
Summary: DFG JIT does not leverage integer speculations on branches
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-14 21:17 PDT by Filip Pizlo
Modified: 2011-09-15 11:19 PDT (History)
0 users

See Also:


Attachments
the patch (4.67 KB, patch)
2011-09-14 21:36 PDT, Filip Pizlo
oliver: review+
webkit-ews: commit-queue-
Details | Formatted Diff | Diff
the patch (4.22 KB, patch)
2011-09-14 22:10 PDT, Filip Pizlo
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Filip Pizlo 2011-09-14 21:17:42 PDT
DFG JIT specializes branches for booleans.  But it does not specialize them for integers.
Comment 1 Filip Pizlo 2011-09-14 21:36:05 PDT
Created attachment 107452 [details]
the patch
Comment 2 Oliver Hunt 2011-09-14 21:50:09 PDT
Comment on attachment 107452 [details]
the patch

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

r+ if you remove the platform.h change

> Source/JavaScriptCore/wtf/Platform.h:960
> -#define ENABLE_TIERED_COMPILATION 0
> +#define ENABLE_TIERED_COMPILATION 1

You really want to commit this don't you?
Comment 3 Early Warning System Bot 2011-09-14 21:52:34 PDT
Comment on attachment 107452 [details]
the patch

Attachment 107452 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/9651923
Comment 4 Filip Pizlo 2011-09-14 22:10:26 PDT
(In reply to comment #2)
> (From update of attachment 107452 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=107452&action=review
> 
> r+ if you remove the platform.h change
> 
> > Source/JavaScriptCore/wtf/Platform.h:960
> > -#define ENABLE_TIERED_COMPILATION 0
> > +#define ENABLE_TIERED_COMPILATION 1
> 
> You really want to commit this don't you?

Yes I really do! :-)
Comment 5 Filip Pizlo 2011-09-14 22:10:55 PDT
Created attachment 107456 [details]
the patch
Comment 6 Filip Pizlo 2011-09-15 00:27:36 PDT
Landed in r95170.
Comment 7 Geoffrey Garen 2011-09-15 11:19:49 PDT
Comment on attachment 107456 [details]
the patch

Clearing r? since this was committed.