[PATCH] D15448: [analyzer] SVal Visitor.
Artem Dergachev via cfe-commits
cfe-commits at lists.llvm.org
Wed Jan 13 09:17:16 PST 2016
NoQ removed rL LLVM as the repository for this revision.
NoQ updated this revision to Diff 44758.
NoQ added a comment.
Reverted the patch due to a few issues. This revision should fix these issues.
The explain-svals test is fixed to target a specific target, in order to make sure that the definition of size_t always agrees with the target triple, otherwise the test would fail (shame on me, should have guessed!).
Not quite sure what to do with the failure of the Modules buildbot (http://lab.llvm.org:8011/builders/clang-x86_64-linux-selfhost-modules/builds/10562/steps/compile.llvm.stage2/logs/stdio). I added the new .def-files to the list of "textual" headers in module.modulemap, but i'm not brave enough to go ahead and commit again and make that sort of thing work through trial and error, as maybe there are more things i need to do.
Fix a small whitespace error introduced by the patch.
Right after committing http://reviews.llvm.org/D16062, i noticed that `MemRegion` itself also doesn't need to be a friend of `MemRegionManager`. Added this to this patch, as they're related, i guess.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 40499 bytes
Desc: not available
More information about the cfe-commits