[clang] [nfc][analyzer] Add MemSpace trait to program state (PR #123003)

Gábor Horváth via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 17 05:22:04 PST 2025


Xazax-hun wrote:

> This expression is indeed error-prone, but the problem is not the use of isa but the use of the method getMemorySpace()

My bad! You are right, it is not the `isa`. :) 

>  this error-prone situation will be resolved soon in a follow-up commit 

I think it would be nice if we never had the error prone APIs to begin with since people can lose interest, or might get hit by a bus and we are left in an unfortunate state. That being said if there is no easy way to make this less error prone I am OK with delaying that to a follow up PR. But might worth exploring if this is something that can be done in a couple hours, why not. 

https://github.com/llvm/llvm-project/pull/123003


More information about the cfe-commits mailing list