[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 13:18:40 PDT 2022


mtrofin added a comment.

In D134013#3796393 <https://reviews.llvm.org/D134013#3796393>, @MaskRay wrote:

> LGTM. I think fixing just ELF is fine, but will be great if you fix COFF/MachO/WebAssembly as well as follow-up patches.

ack

> I'd recommend some options more general and more common than `-mllvm -enable-ml-inliner=default -mllvm -max-devirt-iterations=1`, but using them is fine, 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