[PATCH] Introduce prefetch builtin intrinsics for ARM and AArch64

Yi Kong kongy.dev at gmail.com
Wed Aug 13 12:30:34 PDT 2014


Thanks for reviewing! Committed r215568 and r215569.

-Yi

On 13 August 2014 17:32, Renato Golin <renato.golin at linaro.org> wrote:
> On 13 August 2014 17:28, Yi Kong <kongy.dev at gmail.com> 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



More information about the cfe-commits mailing list