Closed Bug 1239751 Opened 6 years ago Closed 6 years ago

CType's FieldInfoHash does not need a post barrier

Categories

(Core :: JavaScript: GC, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla46
Tracking Status
firefox46 --- fixed

People

(Reporter: terrence, Assigned: terrence)

References

Details

Attachments

(1 file)

It's a string, so it can't move anyway. And even if it did move, the hash only depends on the string contents in this case. So.

This swath of code can come out. That leaves us with two callers of rekeyIfMoved.
Attachment #8707970 - Flags: review?(jcoppeard)
Attachment #8707970 - Flags: review?(jcoppeard) → review+
https://hg.mozilla.org/mozilla-central/rev/4ab1ae8a3379
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.