[PATCH] D57422: [LTO] Set CGOptLevel in LTO config.

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 29 17:11:32 PST 2019


sbc100 added a comment.

I see.    So there should be enough information in each of the bitcode files to set the CGOptLevel accordingly for each function?

The fact that gold saw a performance regression is suspicious no?  Does that mean there is performance gain we can get by adding it here?

Perhaps we should remove the CGOptLevel from the LTO completely once we remove it from gold?   It looks like tools/llvm-lto2/llvm-lto2.cpp also does that same thing BTW.


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D57422





More information about the llvm-commits mailing list