NoQ added inline comments.

================
Comment at: clang/lib/StaticAnalyzer/Core/DynamicType.cpp:42
+  return MR->StripCasts();
+}
+
----------------
Charusso wrote:
> Probably a consistent way of `MemRegion` handling is more appropriate.
Pointer casts are quite a can of worms. I suggest ignoring it for now and 
dealing with it as needed. I.e., your code looks good... for now.




Repository:
  rC Clang

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

https://reviews.llvm.org/D67079



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to