Closed Bug 459151 Opened 16 years ago Closed 16 years ago

TM: non-fused ifs emit different comparison than the interpreter evaluates

Categories

(Core :: JavaScript Engine, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: gal, Unassigned)

Details

Attachments

(1 file)

Attached patch patchSplinter Review
      No description provided.
Attachment #342342 - Flags: review?(danderson)
tests?
Attachment #342342 - Flags: review?(danderson) → review+
http://hg.mozilla.org/tracemonkey/rev/9ea000c6ffb4
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Note: this blocked the thin loop change. No extra coverage needed. Our test suite already covers all cases. This change makes cond in ifop predict correctly whether the branch will be taken or not, which we need for the thin loop handling.
Flags: in-testsuite-
Flags: in-litmus-
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: