[PATCH] D62313: Add a test for "nofree" function attribute

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 12 22:20:55 PDT 2019


jdoerfert added a subscriber: homerdin.
jdoerfert added a comment.

In D62313#1540967 <https://reviews.llvm.org/D62313#1540967>, @uenoku wrote:

> In D62313#1540817 <https://reviews.llvm.org/D62313#1540817>, @jdoerfert wrote:
>
> > As mentioned in D62766 <https://reviews.llvm.org/D62766>, we should make `nofree` an enum attribute. See D62766 <https://reviews.llvm.org/D62766> for some pointer on how to do so.
>
>
> As I see it, D49165 <https://reviews.llvm.org/D49165> is rebased.
>  Which is better to wait for D49165 <https://reviews.llvm.org/D49165> or to make nofree enum attribute in this patch?


Good point. If @homerdin adds the attribute in that patch you do not have to.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62313





More information about the llvm-commits mailing list