[PATCH] D39764: [cfi-verify] Made FileAnalysis operate on a GraphResult rather than build one and validate it.

Mitch Phillips via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 8 14:29:48 PST 2017


hctim updated this revision to Diff 122156.
hctim added a comment.

Made unit tests not use a helper, they should explicitly individually check the result against the expected enum.


https://reviews.llvm.org/D39764

Files:
  test/tools/llvm-cfi-verify/X86/blacklist-expected-unprotected.s
  test/tools/llvm-cfi-verify/X86/blacklist-match-fun.s
  test/tools/llvm-cfi-verify/X86/blacklist-unexpected-protected.s
  tools/llvm-cfi-verify/lib/FileAnalysis.cpp
  tools/llvm-cfi-verify/lib/FileAnalysis.h
  tools/llvm-cfi-verify/llvm-cfi-verify.cpp
  unittests/tools/llvm-cfi-verify/FileAnalysis.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39764.122156.patch
Type: text/x-patch
Size: 18358 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171108/10024349/attachment.bin>


More information about the llvm-commits mailing list