[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