[PATCH] D91327: [NewPM] Redesign of PreserveCFG Checker

Yevgeny Rouban via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 22 22:11:57 PST 2020


yrouban added inline comments.


================
Comment at: llvm/include/llvm/Passes/StandardInstrumentations.h:134
 
 public:
   static cl::opt<bool> VerifyPreservedCFG;
----------------
kuhar wrote:
> yrouban wrote:
> > kuhar wrote:
> > > not necessary anymore
> > there can bee a need to disabled/enable (e.g. for some tests or for debugging).
> I meant the 'public:'. You made everything public at the very top of the class.
sure


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

https://reviews.llvm.org/D91327



More information about the llvm-commits mailing list