[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