Closed
Bug 736419
Opened 12 years ago
Closed 12 years ago
IonMonkey: Code is not traced on ARM
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mjrosenb, Unassigned)
Details
Attachments
(2 files)
3.54 KB,
patch
|
jbramley
:
review+
|
Details | Diff | Splinter Review |
6.20 KB,
patch
|
dvander
:
review+
|
Details | Diff | Splinter Review |
In addition to not always marking code that we branch to, the markers are never copied into the final object, so we never trace through code that is only referenced through jitted code.
Attachment #606514 -
Flags: review?(Jacob.Bramley)
Comment 1•12 years ago
|
||
Comment on attachment 606514 [details] [diff] [review] /home/mrosenberg/patches/moreGCTracing-r0.patch Review of attachment 606514 [details] [diff] [review]: ----------------------------------------------------------------- I can't vouch that the GC tracing is now done correctly because I'm not sure how that works, but the patch looks sane. It'd probably be a good idea to get a second review. ::: js/src/ion/arm/MacroAssembler-arm.cpp @@ +1163,5 @@ > ma_mov(Imm32(0xdeadbeef), ScratchRegister); > #endif > Push(ScratchRegister); // padding > Push(Imm32(descriptor)); // descriptor > // TODO: Use relocation here. Is the TODO still valid?
Attachment #606514 -
Flags: review?(Jacob.Bramley) → review+
Reporter | ||
Comment 2•12 years ago
|
||
It looks like I forgot to land this, so I've updated it with the code to fix our latest v8 woes.
Attachment #612878 -
Flags: review?(dvander)
Comment on attachment 612878 [details] [diff] [review] /home/mrosenberg/patches/moreGCTracing-r1.patch Review of attachment 612878 [details] [diff] [review]: ----------------------------------------------------------------- ::: js/src/ion/arm/MacroAssembler-arm.cpp @@ +1187,5 @@ > #endif > Push(ScratchRegister); // padding > Push(Imm32(descriptor)); // descriptor > // TODO: Use relocation here. > + addPendingJump(m_buffer.nextOffset(), target->raw(), Relocation::IONCODE); It looks like you can remove this :TODO: now. It seems like it would be better, though, to package this (addPendingJump; move; call) sequence into its own helper.
Attachment #612878 -
Flags: review?(dvander) → review+
Reporter | ||
Comment 4•12 years ago
|
||
landed: http://hg.mozilla.org/projects/ionmonkey/rev/572d5f29400e
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•