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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to