Remove now-unnecessary hazard analysis annotations

RESOLVED FIXED in Firefox 50

Status

()

Core
JavaScript: GC
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: sfink, Assigned: sfink)

Tracking

(Blocks: 1 bug)

unspecified
mozilla50
Points:
---

Firefox Tracking Flags

(firefox50 fixed)

Details

Attachments

(1 attachment)

Some of the annotations are no longer necessary.
Created attachment 8767299 [details] [diff] [review]
Remove now-unnecessary hazard analysis annotations

I believe what fixed these are (1) better devirtualization and (2) removal of fallback code in constructors that could only be invoked when exceptions were thrown.
Attachment #8767299 - Flags: review?(terrence)
Comment on attachment 8767299 [details] [diff] [review]
Remove now-unnecessary hazard analysis annotations

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

Nice!
Attachment #8767299 - Flags: review?(terrence) → review+

Comment 3

2 years ago
Pushed by sfink@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/38099407d6e9
Remove now-unnecessary hazard analysis annotations, r=terrence

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/38099407d6e9
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox50: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.