This patch adds -m elf32_x86_64 to lld. But it doesn't generate working
x32 binaries.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Event Timeline
Comment Actions
LGTM modulo nit.
For the future, please upload a diff with context, so that we can review directly on phab.
svn diff --diff-cmd=diff -x-U50000 or git diff -U50000
Thanks for the patch!
ELF/Driver.cpp | ||
---|---|---|
73 ↗ | (On Diff #63690) | This is unsorted, maybe. |
Comment Actions
Can you create working x32 executables with this patch? I expected a small change, but this is smaller than I imagined. For example, it would create PLTs exactly the same as the x86-64 ABI. I'm wondering if it was expected.
Comment Actions
LGTM, but please update the commit message to say that this is an initial patch for x32 and more work is needed to create working x32 binaries.