[PATCH] D79371: [ELF] Add convenience TableGen classes to enforce two dashes for options not supported by GNU ld
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 4 15:05:32 PDT 2020
MaskRay created this revision.
MaskRay added a reviewer: tejohnson.
Herald added subscribers: llvm-commits, arphaman, dexonsmith, steven_wu, hiraditya, arichardson, emaste.
Herald added a reviewer: espindola.
Herald added a project: LLVM.
For many options, we have to support either one or two dash to be
compatible with GNU ld. For some, we don't have compatibility issue can
enforce strict double dashes.
Affected options:
- --thinlto-*
- --lto-*
- --shuffle-sections=
This patch does not change `-plugin-opt=*` because clang driver passes
`-plugin-opt=*` and I don't intend to cause churn.
In 2000, GNU ld tried something similar
https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=e4897a3288f37d5f69e8acd256a6e83e607fe8d8
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D79371
Files:
lld/ELF/Options.td
lld/test/ELF/lto/devirt_vcall_vis_public.ll
lld/test/ELF/lto/thinlto-emit-imports.ll
lld/test/ELF/lto/thinlto-index-file.ll
lld/test/ELF/lto/thinlto-obj-path.ll
lld/test/ELF/lto/thinlto-object-suffix-replace.ll
lld/test/ELF/lto/thinlto-prefix-replace.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D79371.261940.patch
Type: text/x-patch
Size: 11176 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200504/22796c5c/attachment.bin>
More information about the llvm-commits
mailing list