[PATCH] D135014: [lld][ELF] Fix lazy ThinLTO index writing in thin archives

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 4 11:18:13 PDT 2022


MaskRay added inline comments.


================
Comment at: lld/test/ELF/lto/thinlto-emit-index-thin-archive.ll:3
+
+;; Setup
+; RUN: rm -rf %t.dir && mkdir %t.dir && cd %t.dir
----------------
Delete `;; Setup`. It's not useful


================
Comment at: lld/test/ELF/lto/thinlto-emit-index-thin-archive.ll:5
+; RUN: rm -rf %t.dir && mkdir %t.dir && cd %t.dir
+; RUN: mkdir dir1
+; RUN: mkdir dir2
----------------
`mkdir dir1 dir2`


================
Comment at: lld/test/ELF/lto/thinlto-emit-index-thin-archive.ll:40
+
+; CHECK-WHOLE-ARCHIVE: lib.a(unused.o at {{[1-9][0-9]+}})
+
----------------
Rename `CHECK-UNUSED` and just reuse its check since the pattern is exactly the same.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135014



More information about the llvm-commits mailing list