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

Stefan Stipanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 13 08:55:31 PST 2019


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

In D70140#1743354 <https://reviews.llvm.org/D70140#1743354>, @jdoerfert wrote:

> In D70140#1743338 <https://reviews.llvm.org/D70140#1743338>, @arsenm wrote:
>
> > Typo hor in commit message
>
>
> Two typos even. Thanks for noticing!


Thanks! Will double check before I commit



================
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.
 
----------------
jdoerfert wrote:
> NOTE can be removed as well, I think.
I'll remove it before I commit.


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