[PATCH] D134013: [lld][thinlto] Include -mllvm options in the thinlto cache key

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 16 10:24:06 PDT 2022


mtrofin added inline comments.


================
Comment at: lld/test/ELF/lto/cache.ll:55
+; Check that mllvm options participate in the cache key
+; RUN: rm -rf %t.cache
+; RUN: mkdir %t.cache
----------------
MaskRay wrote:
> In the top of the file and other places in test/ELF, we prefer `rm -rf %t.cache && mkdir %t.cache` in one line
ack - let me fix the other occurrence here then, too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134013



More information about the llvm-commits mailing list