Closed
Bug 692903
Opened 13 years ago
Closed 13 years ago
InferSpew should identify the scripts it talks about in more detail
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla10
People
(Reporter: jimb, Unassigned)
Details
(Whiteboard: [inbound])
Attachments
(1 file)
1.87 KB,
patch
|
bhackett1024
:
review+
|
Details | Diff | Splinter Review |
Still more trivia. This patch delays assigning id numbers to scripts until the first time the id is actually retrieved, and at that point produces an InferSpew message identifying the script by id, address, url and line number. This means that we only identify scripts we actually mention in InferSpew output. If JSScript::id had users other than jsinfer, this wouldn't be appropriate. With this patch applied, we get output like this, the first time a given script is mentioned: [infer] script #2: 0x7ffff5906660 /home/jimb/moz/dbg/js/src/jit-test/lib/prolog.js:1 [infer] typeSet: T0x9ff748 bytecode0 #2 ...
Attachment #565622 -
Flags: review?(bhackett1024)
Comment 1•13 years ago
|
||
Comment on attachment 565622 [details] [diff] [review] Identify scripts more helpfully in InferSpew output. The script file/line numbers are also available when the compartment's types get printed out, though that only happens after execution finishes with INFERFLAGS=result/full
Attachment #565622 -
Flags: review?(bhackett1024) → review+
Reporter | ||
Comment 2•13 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/6576946586f4
Status: NEW → ASSIGNED
Whiteboard: [inbound]
https://hg.mozilla.org/mozilla-central/rev/f1f27d2354da
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
You need to log in
before you can comment on or make changes to this bug.
Description
•