[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