Bug 95918 - ASM llint should not use the JIT when the useJIT() option is false
Summary: ASM llint should not use the JIT when the useJIT() option is false
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-09-05 17:42 PDT by Mark Lam
Modified: 2012-09-06 03:12 PDT (History)
3 users (show)

See Also:

Attachments
Fix. (4.28 KB, patch)
2012-09-05 17:48 PDT, Mark Lam 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Lam 2012-09-05 17:42:59 PDT 
r127554 (https://bugs.webkit.org/show_bug.cgi?id=95809) caused a regression where the ASM llint will still the JIT when the useJIT() option is false, but only when useRegExpJIT() (previously useYarrJIT() is true).

Alsop renamed useYarrJIT() to useRegExpJIT().
Comment 1 Mark Lam 2012-09-05 17:48:25 PDT 
Created attachment 162382 [details]
Fix.
Comment 2 Geoffrey Garen 2012-09-05 20:57:33 PDT 
Comment on attachment 162382 [details]
Fix.

r=me
Comment 3 WebKit Review Bot 2012-09-06 03:12:50 PDT 
Comment on attachment 162382 [details]
Fix.

Clearing flags on attachment: 162382

Committed r127719: <http://trac.webkit.org/changeset/127719>
Comment 4 WebKit Review Bot 2012-09-06 03:12:53 PDT 
All reviewed patches have been landed.  Closing bug.