Bug 115998 - fourthTier: FTL shouldn't use FastISel and Small code model should be turned off for now
Summary: fourthTier: FTL shouldn't use FastISel and Small code model should be turned ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Filip Pizlo
URL:
Keywords:
Depends on:
Blocks: 112840
  Show dependency treegraph
 
Reported: 2013-05-12 17:28 PDT by Filip Pizlo
Modified: 2013-05-21 12:43 PDT (History)
7 users (show)

See Also:


Attachments
the patch (2.39 KB, patch)
2013-05-12 17:31 PDT, Filip Pizlo
no flags Details | Formatted Diff | Diff
LLVM patch (503 bytes, patch)
2013-05-12 17:31 PDT, Filip Pizlo
no flags Details | Formatted Diff | Diff
the patch (2.28 KB, patch)
2013-05-21 12:07 PDT, Filip Pizlo
oliver: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Filip Pizlo 2013-05-12 17:28:27 PDT
Patch forthcoming.

This is mostly for corresponding fixes in LLVM.
Comment 1 Filip Pizlo 2013-05-12 17:31:07 PDT
Created attachment 201514 [details]
the patch
Comment 2 Filip Pizlo 2013-05-12 17:31:22 PDT
Created attachment 201515 [details]
LLVM patch
Comment 3 Geoffrey Garen 2013-05-15 15:18:59 PDT
Comment on attachment 201514 [details]
the patch

r=me
Comment 4 Filip Pizlo 2013-05-21 11:16:01 PDT
Comment on attachment 201514 [details]
the patch

Clearing r+ because I'm just going to turn off the small code model, since it doesn't work, and my bugfix isn't being reviewed.
Comment 5 Filip Pizlo 2013-05-21 12:07:46 PDT
Created attachment 202456 [details]
the patch
Comment 6 Filip Pizlo 2013-05-21 12:43:19 PDT
Landed in http://trac.webkit.org/changeset/150470