[PATCH] D120461: [RISCV] Custom lower integer vector multify reduction

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 24 08:59:12 PST 2022


craig.topper added a comment.

I don't think this is the right way to do this. The issue is marked by this FIXME in LowerVECTOR_SHUFFLE

  // If we are doing a SLIDEDOWN+SLIDEUP, reduce the VL for the SLIDEDOWN.   
  // FIXME: If we are only doing a SLIDEDOWN, don't reduce the VL as it      
  // causes multiple vsetvlis in some test cases such as lowering            
  // reduce.mul 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120461



More information about the llvm-commits mailing list