The problem comes from the recent addition of the eager bailing mode, which makes us encode the snapshots before we use the visitor to visit the instruction. The problem is that the snapshot encoding stores information about the last visited instruction. Thus, if the first instruction of the basic block is a bailing instruction, then the debug information will report information about the conditional of the last encoded block.
Created attachment 8765955 [details] [diff] [review] Register instruction information before encoding snapshots. setElement is normally called by the accept function, but as this code has to be executed before we call the accept function (because we need to have the entry register allocation), we have to call it manually to satisfy the use of the "instruction()" method in "CodeGeneratorShared::encode(LSnapshot*)"
Attachment #8765955 - Flags: review?(hv1989)
Attachment #8765955 - Flags: review?(hv1989) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/6ffe64c98d40 Register instruction information before encoding snapshots. r=h4writer
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox50: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.