[PATCH] D63976: Allow clang -Os and -Oz to work with -flto and lld

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 2 03:04:30 PDT 2019


ruiu added a comment.

I agree with Teresa. I don't think automatically setting O3 <https://reviews.llvm.org/owners/package/3/> for Os and Oz is a good idea because these three options are different (that's why we have three different options in the first place). Adding an Os and Oz to lld's LTO option seems like a good idea, but they should be mapped to their corresponding features.


Repository:
  rC Clang

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

https://reviews.llvm.org/D63976





More information about the llvm-commits mailing list