Closed Bug 780860 Opened 12 years ago Closed 12 years ago

IonMonkey: Mark some instructions as movable

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: jandem, Assigned: jandem)

References

(Blocks 1 open bug)

Details

(Whiteboard: [ion:p2])

Attachments

(1 file)

Attached patch PatchSplinter Review
Allows hoisting the Math.abs(j) in gaussian-blur. Not sure if this helps on tip too but it's definitely a small win once we favor OSR for outer loops.
Attachment #649598 - Flags: review?(dvander)
What about sqrt?
(In reply to Tom Schuster [:evilpie] from comment #1)
> What about sqrt?

The patch marks MSqrt as movable too.
Attachment #649598 - Flags: review?(dvander) → review+
https://hg.mozilla.org/projects/ionmonkey/rev/c55e7945d3ab
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.