Thanks for reviewing! Committed r215568 and r215569.

-Yi

On 13 August 2014 17:32, Renato Golin <[email protected]> wrote:
> On 13 August 2014 17:28, Yi Kong <[email protected]> wrote:
>> __pldx is not an obvious name for generating instruction prefetch as
>> well as data prefetching, especially when trying to use the intrinsic
>> directly instead of through the arm_acle header. Also, it is a
>> Clang/GCC convention to have target-specific builtin intrinsic name to
>> start with __builtin_{target name}.
>
> I get your meaning, I dislike it myself, too.
>
> LGTM, thanks!
>
> Looking forward to your other PL* patch.
>
> cheers,
> --renato
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to