[PATCH] D54823: [analyzer][MallocChecker][NFC] Document and reorganize some functions

Kristóf Umann via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 19 13:28:56 PDT 2019


Szelethus marked an inline comment as done.
Szelethus added a comment.

In D54823#1675604 <https://reviews.llvm.org/D54823#1675604>, @NoQ wrote:

> Is it just me or phabricator somehow refuses to display the changes since the last diff here? That's probably the commit diff is involved somehow. So i'm confused what exactly has changed >.<


File name change, I'm using the monorepo now.



================
Comment at: clang/lib/StaticAnalyzer/Checkers/MallocChecker.cpp:3137
+      if (RS->isAllocated() || RS->isAllocatedOfSizeZero())
+        if (!IsConstPointerEscape || checkIfNewOrNewArrayFamily(RS))
+          State = State->set<RegionState>(sym, RefState::getEscaped(RS));
----------------
This was the change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D54823





More information about the llvm-commits mailing list