[PATCH] D38428: Add FileVerifier::isCFIProtected().

Vlad Tsyrklevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 9 09:36:31 PDT 2017


vlad.tsyrklevich added inline comments.


================
Comment at: tools/llvm-cfi-verify/FileAnalysis.h:72
+  // otherwise.
+  bool isCFIProtected(uint64_t Address) const;
+
----------------
`isCFIProtectedIndirectBranch` ?


================
Comment at: tools/llvm-cfi-verify/llvm-cfi-verify.cpp:46
     outs() << "\n";
+    outs() << "  Protected? " << Verifier.isCFIProtected(Address) << "\n";
   }
----------------
Only print this line for indirect branches?


https://reviews.llvm.org/D38428





More information about the llvm-commits mailing list