If an instruction does not override congruentTo, it defaults to comparing the value numbers of the operands. This is error prone, see bug 719774.
I don't really like the name "congruentIfEqualOperands", but I can't think of a better name, suggestions welcome.
Attachment #590770 - Flags: review?(dvander)
Attachment #590770 - Flags: review?(dvander) → review+
(congruentIfOperandsEqual reads 8% better)
(In reply to David Anderson [:dvander] from comment #2) > (congruentIfOperandsEqual reads 8% better) Pushed with this change: https://hg.mozilla.org/projects/ionmonkey/rev/151c3f95b870 Probably a SS win because we're now able to hoist the SETELEM bounds check (the alias set for MStoreElement was too strict).
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.