[llvm] 529b3d1 - [RISCV][TTI] Remove SK_Select from manual splitting in getShuffleCost

[Philip Reames via llvm-commits]

Author: Philip Reames
Date: 2025-02-24T07:53:53-08:00
New Revision: 529b3d16daf2c7970f6f0b1f97e8ed09891c726a

URL: https://github.com/llvm/llvm-project/commit/529b3d16daf2c7970f6f0b1f97e8ed09891c726a
DIFF: https://github.com/llvm/llvm-project/commit/529b3d16daf2c7970f6f0b1f97e8ed09891c726a.diff

LOG: [RISCV][TTI] Remove SK_Select from manual splitting in getShuffleCost

We have general splitting logic for this kind just below, which to my
knowledge is both correct and precise.  Given no test changes, either
a) the adhoc logic works out the same, or b) we have no coverage. I
did not investigate which.

Added: 
    

Modified: 
    llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp b/llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
index 50662a00732da..dfea25e11c0b6 100644
--- a/llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
+++ b/llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
@@ -585,7 +585,6 @@ InstructionCost RISCVTTIImpl::getShuffleCost(TTI::ShuffleKind Kind,
       case TTI::SK_PermuteSingleSrc:
       case TTI::SK_Transpose:
       case TTI::SK_PermuteTwoSrc:
-      case TTI::SK_Select:
         return true;
       }
     };