NoQ added inline comments.

================
Comment at: lib/StaticAnalyzer/Checkers/MallocChecker.cpp:2383-2384
 
+  if (RS == OldRS)
+    return;
+
----------------
Szelethus wrote:
> Hmmm, I guess we return here because if `RegionState` is unchanged, so should 
> be `ReallocPairs` and `FreeReturnValue`, correct?
Yeah, kinda. Deserves a comment, i guess.


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

https://reviews.llvm.org/D54013



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

Reply via email to