[PATCH] D16044: getVariableName() for MemRegion

Gábor Horváth via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 24 09:25:23 PST 2016


xazax.hun added inline comments.

================
Comment at: lib/StaticAnalyzer/Core/MemRegion.cpp:672
@@ +671,3 @@
+    // Combine variable name with indices.
+    VariableName.insert(VariableName.size() - 1, ArrayIndices);
+  }
----------------
nit: this is in fact an append. It is cleaner instead of using insert.


http://reviews.llvm.org/D16044





More information about the cfe-commits mailing list