[PATCH] D18714: Add writeonly IR attribute

Filipe Cabecinhas via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 12 06:53:33 PDT 2016


Ah, of course.
Thank you!
 Filipe

On Tue, Jul 12, 2016 at 2:50 PM, Nicolai Hähnle <nhaehnle at gmail.com> wrote:
> nhaehnle added inline comments.
>
> ================
> Comment at: llvm/trunk/include/llvm/IR/Intrinsics.td:367
> @@ -360,3 +366,3 @@
>                               llvm_i32_ty, llvm_i1_ty],
> -                            [IntrArgMemOnly, NoCapture<0>]>;
> +                            [IntrArgMemOnly, NoCapture<0>, WriteOnly<0>]>;
>
> ----------------
> filcab wrote:
>> Shouldn't memset be `WriteOnly<1>`?
>>
> This is an attribute on a function argument, the number is the number of the function argument. See also the various NoCapture attributes.
>
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D18714
>
>
>


More information about the llvm-commits mailing list