[PATCH] D39525: [cfi-verify] Add blacklist parsing for result filtering.

Vlad Tsyrklevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 3 10:37:16 PDT 2017


vlad.tsyrklevich accepted this revision.
vlad.tsyrklevich added inline comments.
This revision is now accepted and ready to land.


================
Comment at: tools/llvm-cfi-verify/llvm-cfi-verify.cpp:54
 
+    bool CFIProtected = false;
+
----------------
`bool CFIProtected = Analysis.isIndirect...` is a little clearer.


================
Comment at: tools/llvm-cfi-verify/llvm-cfi-verify.cpp:59
+      CFIProtected = true;
     } else {
       outs() << "U ";
----------------
No braces here.


https://reviews.llvm.org/D39525





More information about the llvm-commits mailing list