a.sidorin added inline comments. ================ Comment at: llvm/tools/clang/lib/StaticAnalyzer/Core/MemRegion.cpp:800 @@ -803,1 +799,3 @@ + if (V.is<const VarRegion*>()) + return V.get<const VarRegion*>(); ---------------- Oops. Lines above should stay in the caller function or be refactored. Otherwise, we'll get the subregion of `V.get<const VarRegion *` instead of returning it directly. This looks like a behaviour change. (And this is the only place where return type is not a `const MemSpaceRegion *`, btw). This issue may also obsolete my comments about return type.
http://reviews.llvm.org/D16873 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
