Closed Bug 1151981 Opened 5 years ago Closed 5 years ago

Remove void* markers

Categories

(Core :: JavaScript: GC, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: terrence, Assigned: terrence)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

We still need to be able to mark a generic Cell* in service of the JITs, and we have enough information to do this easily. Marking void* and making the caller deal with figuring out what kind of thing to mark is a pretty terrible interface though.

This is, surprisingly, line length neutral, despite the new infrastructure for shared upcasts not being used extensively yet.
Attachment #8589230 - Flags: review?(jcoppeard)
Comment on attachment 8589230 [details] [diff] [review]
4.10_remove_mark_internal-v0.diff

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

This looks good.

I had a go at doing without the functor adaptor and came up with a slightly different approach.  I'll post the patch and see what you think.
Attachment #8589230 - Flags: review?(jcoppeard) → review+
This makes CallTyped use variadic template arguments to forward everything to the functor so we don't need to put these into a TracerAdaptorFunctor first.
Attachment #8589605 - Flags: feedback?(terrence)
Comment on attachment 8589605 [details] [diff] [review]
call-typed-update

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

Oooo, template var args! I should have thought of that. This looks much better.
Attachment #8589605 - Flags: review+
Attachment #8589605 - Flags: feedback?(terrence)
Attachment #8589605 - Flags: feedback+
https://hg.mozilla.org/mozilla-central/rev/dd24fea25988
Assignee: nobody → terrence
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
I landed a trivial mingw fixup (check for compiler instead of assuming that XP_WIN means MSVC):

https://hg.mozilla.org/integration/mozilla-inbound/rev/598ccd9bc144
You need to log in before you can comment on or make changes to this bug.