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
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
__
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
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
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
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
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
_
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