[llvm] [RISCV] Remove ST->hasVInstructions() from getIntrinsicInstrCost for cttz/ctlz/ctpop. NFC. (PR #154064)

Jim Lin via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 17 22:57:51 PDT 2025


https://github.com/tclin914 created https://github.com/llvm/llvm-project/pull/154064

That isn't necessary if we've checked ST->hasStdExtZvbb().

>From 29091f7f3d4b6d45eb239ff35d505df2ca266a47 Mon Sep 17 00:00:00 2001
From: Jim Lin <jim at andestech.com>
Date: Mon, 18 Aug 2025 13:40:19 +0800
Subject: [PATCH] [RISCV] Remove ST->hasVInstructions() from
 getIntrinsicInstrCost for cttz/ctlz/ctpop. NFC.

That isn't necessary if we've checked ST->hasStdExtZvbb().
---
 llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp b/llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
index af78b3cc2c7ff..85b3059d87da7 100644
--- a/llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
+++ b/llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
@@ -1431,7 +1431,7 @@ RISCVTTIImpl::getIntrinsicInstrCost(const IntrinsicCostAttributes &ICA,
   case Intrinsic::ctlz:
   case Intrinsic::ctpop: {
     auto LT = getTypeLegalizationCost(RetTy);
-    if (ST->hasVInstructions() && ST->hasStdExtZvbb() && LT.second.isVector()) {
+    if (ST->hasStdExtZvbb() && LT.second.isVector()) {
       unsigned Op;
       switch (ICA.getID()) {
       case Intrinsic::cttz:



More information about the llvm-commits mailing list