[PATCH] D37978: [analyzer] Fix an assertion fail in VirtualCallChecker

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 20 02:10:26 PDT 2017


alexfh added a comment.

Actually, after looking at the getBestDynamicClassType() implementation, I'm pretty sure it does all things IgnoreImpCasts() used to do that are relevant to the use case. So the patch seems to be trivially correct. Other reviewers' comments (if any) can be addressed post-commit. Thanks again for the fix!


Repository:
  rL LLVM

https://reviews.llvm.org/D37978





More information about the cfe-commits mailing list