[PATCH] D13519: [MemCpyOpt] Fix wrong merging of adjacent nontemporal stores into memset calls.

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 8 09:53:09 PDT 2015


qcolombet accepted this revision.
qcolombet added a comment.
This revision is now accepted and ready to land.

Hi Andrea,

>   I am under the impression that a change like that would only have the effect of forcing the backend to expand !nontemporal memsets back to sequences of store instructions.


That’s a good point.
Add that in the comment so that we remember why we made this design decision :).

LGTM.

Thanks,
Q.


http://reviews.llvm.org/D13519





More information about the llvm-commits mailing list