Closed
Bug 1401005
Opened 7 years ago
Closed 7 years ago
lld - elfhack test fails
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox58 fixed)
VERIFIED
FIXED
mozilla58
Tracking | Status | |
---|---|---|
firefox58 | --- | fixed |
People
(Reporter: Sylvestre, Assigned: glandium)
References
Details
Attachments
(4 files)
Follow up of bug 1391183:
[ $(objdump -R test-ctors.so.bak | wc -l) -gt $(objdump -R test-ctors.so | wc -l) ]
# Will either crash or return exit code 1 if elfhack is broken
LD_PRELOAD=/data/jenkins/workspace/firefox-clang-lld-last/obj-x86_64-pc-linux-gnu/build/unix/elfhack/test-array.so /data/jenkins/workspace/firefox-clang-lld-last/obj-x86_64-pc-linux-gnu/build/unix/elfhack/dummy
FAIL
Reporter | ||
Comment 1•7 years ago
|
||
Reporter | ||
Comment 2•7 years ago
|
||
Reporter | ||
Comment 3•7 years ago
|
||
Assignee | ||
Comment 4•7 years ago
|
||
(In reply to Sylvestre Ledru [:sylvestre] from comment #0)
> Follow up of bug 1391183:
>
> [ $(objdump -R test-ctors.so.bak | wc -l) -gt $(objdump -R test-ctors.so |
> wc -l) ]
> # Will either crash or return exit code 1 if elfhack is broken
> LD_PRELOAD=/data/jenkins/workspace/firefox-clang-lld-last/obj-x86_64-pc-
> linux-gnu/build/unix/elfhack/test-array.so
> /data/jenkins/workspace/firefox-clang-lld-last/obj-x86_64-pc-linux-gnu/build/
> unix/elfhack/dummy
> FAIL
You'll note it explicitly says "FAIL". It's *not* crashing.
Summary: lld - dummy binary with the elfhack segsfaults - try 3 → lld - elfhack test fails
Comment hidden (mozreview-request) |
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → mh+mozilla
Comment 6•7 years ago
|
||
mozreview-review |
Comment on attachment 8910568 [details]
Bug 1401005 - Handle the case where the relocation addend is not found at the relocation location.
https://reviewboard.mozilla.org/r/182010/#review187564
Attachment #8910568 -
Flags: review?(nfroyd) → review+
Pushed by nfroyd@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c58ffaa1e410
Handle the case where the relocation addend is not found at the relocation location. r=froydnj
Comment 8•7 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox58:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Reporter | ||
Comment 9•7 years ago
|
||
I will test that but I am blocked by bug 1402362 for now (not related)
Reporter | ||
Comment 10•7 years ago
|
||
Verified, we can build with lld without --disable-elf-hack \o/
Status: RESOLVED → VERIFIED
Reporter | ||
Updated•7 years ago
|
status-firefox57:
affected → ---
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•