[clang] [nfc][analyzer] Add MemSpace trait to program state (PR #123003)
Michael Flanders via cfe-commits
cfe-commits at lists.llvm.org
Thu Jan 16 12:53:11 PST 2025
Flandini wrote:
> I like the direction of this PR but at the same time it makes memspaces a bit more error prone to use. Do you think we could find a way to prevent using `isa` on memspaces?
Do you mean moving away from a class hierarchy definition for memspaces and towards something else, maybe a enum Kind definition, or maybe just try to hide all the casting behind a different API? IIUC, from previous chat with @steakhal, the eventual goal is to get rid of the MemSpaceRegion part of the MemRegion class hierarchy and replace it with something else.
https://github.com/llvm/llvm-project/pull/123003
More information about the cfe-commits
mailing list