[PATCH] D20358: [Clang][AVX512][Intrinsics]Convert AVX non-temporal store builtins to LLVM-native IR.

2016-11-15 Thread Simon Pilgrim via cfe-commits
RKSimon added a comment. Abandon this patch? We replaced the x86 vector non-temporal store builtins with __builtin_nontemporal_store directly in the headers. https://reviews.llvm.org/D20358 ___ cfe-commits mailing list cfe-commits@lists.llvm.org ht

Re: [PATCH] D20358: [Clang][AVX512][Intrinsics]Convert AVX non-temporal store builtins to LLVM-native IR.

2016-06-13 Thread Simon Pilgrim via cfe-commits
RKSimon added a comment. http://reviews.llvm.org/D21272 has now been committed, which I think removes the need for this patch. http://reviews.llvm.org/D20359 is still needed (with the additional tests requested by Craig). http://reviews.llvm.org/D20358 __

Re: [PATCH] D20358: [Clang][AVX512][Intrinsics]Convert AVX non-temporal store builtins to LLVM-native IR.

2016-06-12 Thread Simon Pilgrim via cfe-commits
RKSimon added a comment. In http://reviews.llvm.org/D20358#446241, @RKSimon wrote: > In http://reviews.llvm.org/D20358#446220, @ab wrote: > > > In http://reviews.llvm.org/D20358#446218, @ab wrote: > > > > > In http://reviews.llvm.org/D20358#446210, @RKSimon wrote: > > > > > > > Is there any reaso

Re: [PATCH] D20358: [Clang][AVX512][Intrinsics]Convert AVX non-temporal store builtins to LLVM-native IR.

2016-06-01 Thread Simon Pilgrim via cfe-commits
RKSimon added a comment. In http://reviews.llvm.org/D20358#446220, @ab wrote: > In http://reviews.llvm.org/D20358#446218, @ab wrote: > > > In http://reviews.llvm.org/D20358#446210, @RKSimon wrote: > > > > > Is there any reason why we can't just get rid of all the SSE movnt > > > builtins and use

Re: [PATCH] D20358: [Clang][AVX512][Intrinsics]Convert AVX non-temporal store builtins to LLVM-native IR.

2016-06-01 Thread Ahmed Bougacha via cfe-commits
ab added a comment. In http://reviews.llvm.org/D20358#446218, @ab wrote: > In http://reviews.llvm.org/D20358#446210, @RKSimon wrote: > > > Is there any reason why we can't just get rid of all the SSE movnt builtins > > and use __builtin_nontemporal_store instead > > (http://reviews.llvm.org/D12

Re: [PATCH] D20358: [Clang][AVX512][Intrinsics]Convert AVX non-temporal store builtins to LLVM-native IR.

2016-06-01 Thread Ahmed Bougacha via cfe-commits
ab added a subscriber: ab. ab added a comment. In http://reviews.llvm.org/D20358#446210, @RKSimon wrote: > Is there any reason why we can't just get rid of all the SSE movnt builtins > and use __builtin_nontemporal_store instead (http://reviews.llvm.org/D12313)? I wanted to suggest that too, b

Re: [PATCH] D20358: [Clang][AVX512][Intrinsics]Convert AVX non-temporal store builtins to LLVM-native IR.

2016-06-01 Thread Simon Pilgrim via cfe-commits
RKSimon added a subscriber: RKSimon. RKSimon added a comment. Is there any reason why we can't just get rid of all the SSE movnt builtins and use __builtin_nontemporal_store instead (http://reviews.llvm.org/D12313)? http://reviews.llvm.org/D20358 _

Re: [PATCH] D20358: [Clang][AVX512][Intrinsics]Convert AVX non-temporal store builtins to LLVM-native IR.

2016-05-23 Thread Craig Topper via cfe-commits
craig.topper accepted this revision. craig.topper added a comment. This revision is now accepted and ready to land. LGTM http://reviews.llvm.org/D20358 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/lis