Closed Bug 1043898 Opened 7 years ago Closed 7 years ago

MIR: Remove a few getOperand(0) useless calls in some unary instructions

Categories

(Core :: JavaScript Engine: JIT, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla34

People

(Reporter: bbouvier, Assigned: bbouvier)

Details

Attachments

(2 files)

No description provided.
It sounds to me that "operand" isn't more eloquent than "input", so let's just
use input instead.
Attachment #8462486 - Flags: review?(jdemooij)
Again, num() isn't really eloquent and is used only in one place for MRound, MFloor and MCeil.

MComputeThis overrides input(), which isn't caught by the compiler but is unnecessary.
Attachment #8462488 - Flags: review?(jdemooij)
Attachment #8462486 - Flags: review?(jdemooij) → review+
Attachment #8462488 - Flags: review?(jdemooij) → review+
https://hg.mozilla.org/mozilla-central/rev/182850749379
https://hg.mozilla.org/mozilla-central/rev/43cce527da0e
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
You need to log in before you can comment on or make changes to this bug.