Closed Bug 1018290 Opened 5 years ago Closed 5 years ago

Add back IonScriptCounts to asm.js modules

Categories

(Core :: JavaScript Engine: JIT, defect)

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla32

People

(Reporter: bhackett, Assigned: bhackett)

References

Details

Attachments

(1 file)

Attached patch patchSplinter Review
Bug 998490 removed in rev 4ffd7defeba7 this but it is still necessary to get detailed information about the generated LIR and assembly which executes in asm.js code.  The attached patch backs out the above rev and also cleans things up a bit so that off thread Ion compilation can be used when profiling script execution.
Attachment #8431659 - Flags: review?(luke)
Attachment #8431659 - Flags: review?(luke) → review+
Oops, sorry, for some reason those tests didn't fail on my computer.

https://hg.mozilla.org/integration/mozilla-inbound/rev/1c9fed1fd66c
https://hg.mozilla.org/mozilla-central/rev/1c9fed1fd66c
Assignee: nobody → bhackett1024
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
Depends on: 1019322
You need to log in before you can comment on or make changes to this bug.