[llvm] [RISCV][RFC] Add additional opcodes to RISCVDAGToDAGISel::hasAllNBitUsers (PR #147728)
    Craig Topper via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Jul 21 10:24:55 PDT 2025
    
    
  
topperc wrote:
> If there should be any particular threshold for adding things to the ISel hasAllNBitUsers or we should just keep it in sync best effort (as this PR tries to do)
My primary concern is the compile time cost of the recursion. If we can't construct a test case that shows benefit then I'm not sure we should pay that cost.
https://github.com/llvm/llvm-project/pull/147728
    
    
More information about the llvm-commits
mailing list