[PATCH] D23014: [analyzer] Model base to derived casts more precisely.

Gábor Horváth via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 2 06:35:23 PDT 2016


xazax.hun added inline comments.

================
Comment at: lib/StaticAnalyzer/Core/ExprEngineC.cpp:423
@@ +422,3 @@
+
+        if (!val.isZeroConstant()) {
+          val = getStoreManager().evalDynamicCast(val, T, Failed);
----------------
NoQ wrote:
> I guess if `val` is a //non-zero// constant, it wouldn't make much difference.
I might be wrong, but isn't the only valid constant value for a pointer the zero constant?


https://reviews.llvm.org/D23014





More information about the cfe-commits mailing list