Closed Bug 539876 Opened 13 years ago Closed 13 years ago

nanojit: optimize away 'y = mul x, 1'

Categories

(Core Graveyard :: Nanojit, defect)

Product:
Core Graveyard
Component:
Nanojit
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: n.nethercote, Assigned: n.nethercote)

References

Details

(Whiteboard: fixed-in-nanojit, fixed-in-tracemonkey)

Attachments

(1 file)

patch
762 bytes, patch
rreitmai
: review+
Details | Diff | Splinter Review
Attached patch patch — — Splinter Review 
ExprFilter is pretty aggressive, but it misses this one.  It shows up in TM code sometimes because we index into a table like this:

  n * sizeof(JSTraceType)

and sizeof(JSTraceType) == 1.
Attachment #421776 - Flags: review?(rreitmai)
Attachment #421776 - Flags: review?(rreitmai) → review+
http://hg.mozilla.org/tracemonkey/rev/b831915b57af
Whiteboard: fixed-in-nanojit → fixed-in-nanojit, fixed-in-tracemonkey
http://hg.mozilla.org/mozilla-central/rev/b831915b57af
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.