xazax.hun added a comment.

If the AST is hard to work with would it make sense to try to change the AST a 
bit?



================
Comment at: clang/lib/StaticAnalyzer/Core/CallEvent.cpp:944
+  if (Data) {
+    loc::MemRegionVal MV(static_cast<const MemRegion *>(Data));
+    if (SymbolRef Sym = MV.getAsSymbol(true))
----------------
Maybe reusing `getCXXThisVal` here to reduce the number of casts?


Repository:
  rC Clang

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

https://reviews.llvm.org/D74735



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

Reply via email to