[PATCH] D55898: [x86] add movddup specialization for build vector lowering (PR37502)
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 20 09:29:50 PST 2018
spatel added a comment.
Proposal for load folding improvement is here:
D55936 <https://reviews.llvm.org/D55936>
Also, I reviewed the diffs from:
https://bugs.llvm.org/show_bug.cgi?id=32558
...which was a perf reason for reverting the earlier and more general fix. This patch won't hit that same problem because we are checking that the 2 operands are different (if all 4 operands are the same, then we should defer that to more general splat lowering). AFAICT, the code for that benchmark is going to improve via this fix + the load folding...at least it looks better on paper, no telling what will happen in practice. :)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55898/new/
https://reviews.llvm.org/D55898
More information about the llvm-commits
mailing list