Closed Bug 1820855 Opened 2 years ago Closed 2 years ago

[hazards] documentation and cleanup refactorings

Categories

(Core :: JavaScript: GC, task, P2)

task

Tracking

()

RESOLVED FIXED
114 Branch
Tracking Status
firefox114 --- fixed

People

(Reporter: sfink, Assigned: sfink)

References

(Blocks 1 open bug)

Details

Attachments

(3 files)

I have a small pile of stuff that ended up not being part of any significant change. There's the beginning of a document on the sixgill output format. There's a "simplfication" to the always-confusing BFS_upwards() stuff, which was necessary for a change that turned out to produce too many false positives, but I think the preparatory change is probably still better than what is there now? And finally, there's a pure refactoring that makes some direct data structure manipulation slightly easier to follow.

Blocks: 1820858
Assignee: nobody → sphink
Status: NEW → ASSIGNED

This was necessary for a change that I ended up not needing, but the main point of this patch is the commentary.

Attachment #9321677 - Attachment description: Bug 1820855 - [hazards] Add in the start of documentation of the CFG format. → Bug 1820855 - [hazards] Document the CFG format.
Pushed by sfink@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8603340fd8f4 [hazards] Document the CFG format. r=jonco https://hg.mozilla.org/integration/autoland/rev/8c9713a87ee1 [hazards] Make BFS_upwards a little simpler with respect to reaching the entry point, and comment. r=jonco https://hg.mozilla.org/integration/autoland/rev/7dd4ee29d877 [hazards] Extract out a function matcher r=jonco
Pushed by sfink@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3b3ebbf22ea9 [hazards] Document the CFG format. r=jonco https://hg.mozilla.org/integration/autoland/rev/d5babfab79d6 [hazards] Make BFS_upwards a little simpler with respect to reaching the entry point, and comment. r=jonco https://hg.mozilla.org/integration/autoland/rev/7fb2661a3faf [hazards] Extract out a function matcher r=jonco
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 114 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: