[PATCH] D86420: [DAGCombiner] allow store merging non-i8 truncated ops
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 26 05:17:25 PDT 2020
spatel added a comment.
In D86420#2238310 <https://reviews.llvm.org/D86420#2238310>, @RKSimon wrote:
> LGTM - would you be able to get the "pair swap" rotate tweak done as a followup? Ideally with x86 test coverage as well.
Yes, that doesn't seem like a big add-on. We allowed something like that in D83567 <https://reviews.llvm.org/D83567>.
Also, this code sits outside of all of the other store merging that we do in DAGCombiner, so there's potential for refactoring.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D86420/new/
https://reviews.llvm.org/D86420
More information about the llvm-commits
mailing list