[PATCH] D95307: [StaticAnalyzer] Add checking for degenerate base class in MemRegion

Deep Majumder via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 1 06:53:29 PST 2021


RedDocMD added a comment.

In D95307#2533259 <https://reviews.llvm.org/D95307#2533259>, @vsavchenko wrote:

> I don't know if this is useful or not, but pointer-to-members used to be `void *` up until very recently.  Here is my change on that topic (and it probably where the bug you're trying to fix initiated): https://reviews.llvm.org/D85817
> I thought, it can be useful to see the areas of code where it all happens.

Thank you @vsavchenko. I had actually seen this review by examining the file blames. I don't think the problem is in your review. The trouble is that when the CXXBaseSpecifier list in PointerToMemberData is initialized, it receives a list. That list I suspect contains passed in the base class of the struct. But this list is supposed to be for following static casts. I have to localize that logic.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95307/new/

https://reviews.llvm.org/D95307



More information about the cfe-commits mailing list