[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
Tue Nov 7 15:08:55 PST 2017


hctim created this revision.

Refactors the behaviour of building graphs out of FileAnalysis, allowing for analysis of the GraphResult by the callee without having to rebuild the graph. Means when we want to analyse the constructed graph (planned for later revisions), we don't do repeated work.

Also makes CFI verification in FileAnalysis now return an enum that allows us to differentiate **why** something failed, not just that it did/didn't fail.


https://reviews.llvm.org/D39764

Files:
  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.121994.patch
Type: text/x-patch
Size: 12811 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171107/db2d771f/attachment.bin>


More information about the llvm-commits mailing list