Closed Bug 1039667 Opened 10 years ago Closed 10 years ago

DCE folded MBoundsCheck instructions

Categories

(Core :: JavaScript Engine: JIT, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla33

People

(Reporter: sunfish, Assigned: sunfish)

References

Details

Attachments

(1 file)

With bug 1031607, MBoundsCheck instructions can be folded away, exposing subsequent constant folding. However, the unused MBoundsCheck instructions are currently left in place because they have the isGuard() flag set. Attached is a patch which clears the isGuard flag from MBoundsCheck instructions when they are folded, which allows GVN to delete them.
Attachment #8457061 - Flags: review?(jdemooij)
Comment on attachment 8457061 [details] [diff] [review]
dce-boundscheck.patch

Review of attachment 8457061 [details] [diff] [review]:
-----------------------------------------------------------------

Makes sense.
Attachment #8457061 - Flags: review?(jdemooij) → review+
https://hg.mozilla.org/mozilla-central/rev/f199144be062
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: