Following D109516, this patch re-uses the new helper function for ELF relocation traversal in the x86-64 backend.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Time | Test | |
---|---|---|
300 ms | x64 debian > ORC-x86_64-linux.TestCases/Linux/x86-64::trivial-tls.S Script:
--
: 'RUN: at line 1'; /var/lib/buildkite-agent/builds/llvm-project/build/./bin/clang -m64 -c -o /var/lib/buildkite-agent/builds/llvm-project/build/projects/compiler-rt/test/orc/X86_64LinuxConfig/TestCases/Linux/x86-64/Output/trivial-tls.S.tmp /var/lib/buildkite-agent/builds/llvm-project/compiler-rt/test/orc/TestCases/Linux/x86-64/trivial-tls.S
|
Event Timeline
Comment Actions
The patch aims to be NFC (except debug output format). Please ping me if I introduced a functional change by accident.
llvm/lib/ExecutionEngine/JITLink/ELF_x86_64.cpp | ||
---|---|---|
371 | This causes a warning when I do a release build with clang 10.0.0: ELF_x86_64.cpp:308:13: warning: enumeration value 'PCRel32TLV' not handled in switch [-Wswitch] |
llvm/lib/ExecutionEngine/JITLink/ELF_x86_64.cpp | ||
---|---|---|
371 | I'm also seeing this and build with -Werror |
llvm/lib/ExecutionEngine/JITLink/ELF_x86_64.cpp | ||
---|---|---|
371 | Looks like a bad merge with https://reviews.llvm.org/D109293 (thanks @bkramer for pointing this out). I'll send a fix |