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

Reply via email to