[PATCH] D39750: [cfi-verify] Adds blacklist blame behaviour to cfi-verify.

Vlad Tsyrklevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 8 15:59:57 PST 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:103
+    for (auto &K : {"cfi-icall", "cfi-vcall"}) {
+      for (auto &V :
+           {std::make_pair("src", StringRef(LineInfo.FileName)),
----------------
Could we just unroll this loop? Seems a little clearer and it should be shorter too.


https://reviews.llvm.org/D39750





More information about the llvm-commits mailing list