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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits