[PATCH] D150333: [DAGCombiner][AArch64][VE] Teach BuildUDIV/SDIV to use 2x mul when mulh/mul_lohi are not available.
    Simon Pilgrim via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri May 12 07:19:09 PDT 2023
    
    
  
RKSimon added a comment.
SGTM - probably need to wait for VE review though
================
Comment at: llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp:5996
+      unsigned Size = VT.getSizeInBits();
+      EVT WideVT = EVT::getIntegerVT(*DAG.getContext(), Size * 2);
+      if (isOperationLegal(ISD::MUL, WideVT)) {
----------------
We now have a few cases where we're doing this - we already have widenIntegerVectorElementType but for some reason don't have an equivalent widenIntegerScalarType
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D150333/new/
https://reviews.llvm.org/D150333
    
    
More information about the llvm-commits
mailing list