[PATCH] D138736: [PowerPC] Fix vperm codegen

Maryam Moghadas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 29 10:18:53 PST 2022


maryammo added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:10261
+    }
+    if (V2HasXXSWAPD) {
+      dl = SDLoc(V2->getOperand(0));
----------------
stefanp wrote:
> Is it possible to have a swap on both sides?
I have not see such a case.
btw, this patch is to address the vperm issue, so we can look at the swap on both sides for xxperm if needed separately. Thanks. 


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138736/new/

https://reviews.llvm.org/D138736



More information about the llvm-commits mailing list