[PATCH] D72547: [llvm] Make new pass manager's OptimizationLevel a class

Wed Jan 15 22:05:39 PST 2020

tejohnson accepted this revision.
tejohnson added a comment.
This revision is now accepted and ready to land.

LGTM

One thing to consider changing/removing in the summary is this comment:
"For example, (enum) "Level > 1" captures not only O2 <https://reviews.llvm.org/owners/package/2/> and O3 <https://reviews.llvm.org/owners/package/3/>, but also Os, and Oz."
since that is actually correct (Os/Oz should be included in Level>1 as they are O2 <https://reviews.llvm.org/owners/package/2/>).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72547