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

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 1 04:10:54 PDT 2021


skatkov added a comment.

First iteration: style comments.



================
Comment at: llvm/include/llvm/Passes/StandardInstrumentations.h:124
       if (BBGuards)
         for (auto &BB : *BBGuards) {
           if (BB.second.isPoisoned())
----------------
redundant {}
consider using any_of


================
Comment at: llvm/lib/Passes/StandardInstrumentations.cpp:1017
 
+struct PreservedCFGCheckerAnalysis
+    : public AnalysisInfoMixin<PreservedCFGCheckerAnalysis> {
----------------
Describe the idea of this analisys


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

https://reviews.llvm.org/D91327



More information about the llvm-commits mailing list