[PATCH] D95541: Support Os or Oz inside the LTO

Mon Apr 26 16:51:18 PDT 2021

jinlin added a comment.

In D95541#2718205 <https://reviews.llvm.org/D95541#2718205>, @MaskRay wrote:

> I don' think mapping 's' to 4 and 'z' to 5 is suitable. It is quite different. If you just want `clang -fuse-ld=lld -Oz -flto{,=thin}` to work, it already does after D79919 <https://reviews.llvm.org/D79919>

First of all, the changes in D719919 does not pass the SizeLevel information to any optimizations including inlining.

Secondly, the changes in D79919 <https://reviews.llvm.org/D79919> is not general. It does not work in MacOS.

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

https://reviews.llvm.org/D95541