[PATCH] D95541: Support Os or Oz during LTO

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 27 11:52:33 PST 2021


fhahn requested changes to this revision.
fhahn added a comment.
This revision now requires changes to proceed.

Thanks for the patch.

I think this probably should be split up in 2 different changes:

1. Changes to the pipeline (`llvm/lib/Transforms/IPO/PassManagerBuilder.cpp`). Ideally those come with some data motivating the change and why there won't be regressions.
2. Changes to LTOCodeGenerator & co. They also need a test.


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

https://reviews.llvm.org/D95541



More information about the llvm-commits mailing list