[PATCH] D70140: Use nofree argument attributr hor heap-to-stack conversion

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 12 17:40:22 PST 2019


jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

LGTM, one nit.



================
Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:4158
       // NOTE: Right now, if a function that has malloc pointer as an argument
       // frees memory, we assume that the malloc pointer is freed.
 
----------------
NOTE can be removed as well, I think.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70140





More information about the llvm-commits mailing list