Closed
Bug 1479963
Opened 6 years ago
Closed 6 years ago
Missing callgraph edges due to unhandled constructs
Categories
(Core :: JavaScript: GC, enhancement, P2)
Core
JavaScript: GC
Tracking
()
RESOLVED
FIXED
People
(Reporter: sfink, Assigned: sfink)
References
(Blocks 1 open bug)
Details
Attachments
(3 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
bhackett1024
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
bhackett1024
:
review+
|
Details | Diff | Splinter Review |
sixgill was choking on a few more things and discarding CFGs.
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Comment 2•6 years ago
|
||
I ran into both of these things at important places (triggering a false positive hazard because it couldn't follow GC suppression through the callgraph).
Attachment #8996513 -
Flags: review?(bhackett1024)
Assignee | ||
Comment 3•6 years ago
|
||
IL_IsDestructor was returning true for things like std::forward<Foo::~Foo::<lambda()>> because it saw a "::~" substring in something that isn't even a method, let alone a destructor.
Attachment #8996514 -
Flags: review?(bhackett1024)
Updated•6 years ago
|
Attachment #8996513 -
Flags: review?(bhackett1024) → review+
Updated•6 years ago
|
Attachment #8996514 -
Flags: review?(bhackett1024) → review+
Assignee | ||
Updated•6 years ago
|
Priority: -- → P2
Assignee | ||
Comment 4•6 years ago
|
||
http://hg.mozilla.org/users/sfink_mozilla.com/sixgill/rev/bf2c5b0be0cd
http://hg.mozilla.org/users/sfink_mozilla.com/sixgill/rev/a20bd5870d20
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•