[PATCH] D67782: [ELF] accept thinlto options without --plugin-opt= prefix

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 19 19:12:19 PDT 2019


MaskRay added inline comments.


================
Comment at: lld/test/ELF/lto/thinlto-emit-imports.ll:50
+; RUN: ld.lld -thinlto-index-only -thinlto-emit-imports-files -shared %t1.o %t2.o %t3.o -o %t4
+; RUN: cat %t1.o.imports | count 1
+; RUN: cat %t1.o.imports | FileCheck %s --check-prefix=IMPORTS1
----------------
```
count 1 < %t1.o.imports
FileCheck %s --check-prefix=IMPORTS1 < %t1.o.imports
```

or

```
FileCheck %s --check-prefix=IMPORTS1 --implicit-check-not {{.}} < %t1.o.imports
```


================
Comment at: lld/test/ELF/lto/thinlto-index-file.ll:16
+; Check that this also works without the --plugin-opt= prefix.
+; RUN: ld.lld -thinlto-index-only=%t.idx -shared %t1.o %t2.o %t3.o -o %t4
+; RUN: FileCheck %s < %t.idx
----------------
`%t4` -> `/dev/null`


================
Comment at: lld/test/ELF/lto/thinlto-obj-path.ll:15
+; RUN: rm -f %t4.o
+; RUN: ld.lld -thinlto-index-only -lto-obj-path=%t4.o -shared %t1.o %t2.o -o %t3
+; RUN: llvm-readobj -h %t4.o | FileCheck %s
----------------
`%t3` -> `/dev/null`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67782/new/

https://reviews.llvm.org/D67782





More information about the llvm-commits mailing list