Closed
Bug 695712
Opened 13 years ago
Closed 1 year ago
ARM xptcstubs assembly symbols tweaks
Categories
(Core :: XPCOM, defect)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: glandium, Assigned: glandium)
Details
Attachments
(1 file)
3.44 KB,
patch
|
Details | Diff | Splinter Review |
The ARM xptcstubs symbols all end up in the .text section even with -ffunction-sections, and have a size of 0, which makes them invisible to e.g. valgrind.
Assignee | ||
Comment 1•13 years ago
|
||
PoC. This will need update against bug 681602.
Assignee | ||
Comment 2•13 years ago
|
||
Forgot to mention: since individual stubs use "b" to jump to the SharedStub, they all need to be kept close to each other, because moving one of them too far away from the others can break everything. I think there's an actual bug in gold that is responsible for the breakage, though.
Assignee | ||
Comment 3•13 years ago
|
||
(In reply to Mike Hommey [:glandium] from comment #2)
> Forgot to mention: since individual stubs use "b" to jump to the SharedStub,
> they all need to be kept close to each other, because moving one of them too
> far away from the others can break everything. I think there's an actual bug
> in gold that is responsible for the breakage, though.
It actually looks like gold is doing the right thing. Maybe I was having a separate issue.
Comment 4•13 years ago
|
||
It's not the lack of a size that confuses V; it's the lack of a type.
As it stands, these fns get listed as STT_NOTYPE and are ignored by V.
Similar comments apply to bug 695719.
Even without this patch or the one in bug 695719, valgrind rev 12248
reduces the number of non-identified libxul.so basic blocks from 41 out
of circa 85000 down to just 2.
Updated•2 years ago
|
Severity: normal → S3
Assignee | ||
Updated•1 year ago
|
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•