[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