[PATCH] D81223: Size LTO (1/3): Standardizing the use of OptimizationLevel

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 11 08:47:05 PDT 2020


tejohnson added a comment.

Sorry I haven't had a chance to review this yet. Added @mtrofin who recently added the OptimizationLevel class that this is leveraging, and @yamauchi who has also been looking at size optimizations in llvm.

At a first glance it isn't clear to me how much of this is NFC related refactoring/cleanup vs behavior change. If it has both, it would be helpful to split into an NFC patch first and then a follow on with actual behavior changes. Can you clarify?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81223





More information about the llvm-commits mailing list