Bug 20419 - Remove op_jless
Summary: Remove op_jless
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Cameron Zwarich (cpst)
URL:
Keywords:
: 19823 (view as bug list)
Depends on:
Blocks:
 
Reported: 2008-08-17 13:01 PDT by Cameron Zwarich (cpst)
Modified: 2008-08-18 07:39 PDT (History)
0 users

See Also:


Attachments
Proposed patch (3.89 KB, patch)
2008-08-17 13:03 PDT, Cameron Zwarich (cpst)
mjs: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Cameron Zwarich (cpst) 2008-08-17 13:01:41 PDT
After we added the loop opcodes, I wanted to remove op_jless, as we now have op_loop_if_less. It will only be used when less is inside of a logical operator, which doesn't really justify its continued existence. Removing it was originally a random slowdown on SunSpider, but now it's a 0.1% speedup.
Comment 1 Cameron Zwarich (cpst) 2008-08-17 13:03:43 PDT
Created attachment 22845 [details]
Proposed patch
Comment 2 Maciej Stachowiak 2008-08-17 14:07:19 PDT
Comment on attachment 22845 [details]
Proposed patch

r=me
Comment 3 Cameron Zwarich (cpst) 2008-08-17 14:09:31 PDT
*** Bug 19823 has been marked as a duplicate of this bug. ***
Comment 4 Cameron Zwarich (cpst) 2008-08-18 07:39:48 PDT
Landed in r35810.