[llvm] [NVPTX][NFC] Rename hasAAFeatures() (PR #127990)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 20 03:15:58 PST 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-nvptx
Author: Durgadoss R (durga4github)
<details>
<summary>Changes</summary>
This patch renames hasAAFeatures() to hasArchAccelFeatures()
and updates its usages in tablegen files.
---
Full diff: https://github.com/llvm/llvm-project/pull/127990.diff
3 Files Affected:
- (modified) llvm/lib/Target/NVPTX/NVPTXInstrInfo.td (+1-1)
- (modified) llvm/lib/Target/NVPTX/NVPTXIntrinsics.td (+1-1)
- (modified) llvm/lib/Target/NVPTX/NVPTXSubtarget.h (+3-3)
``````````diff
diff --git a/llvm/lib/Target/NVPTX/NVPTXInstrInfo.td b/llvm/lib/Target/NVPTX/NVPTXInstrInfo.td
index 7d9697e40e6ab..5fc0add89d7cb 100644
--- a/llvm/lib/Target/NVPTX/NVPTXInstrInfo.td
+++ b/llvm/lib/Target/NVPTX/NVPTXInstrInfo.td
@@ -142,7 +142,7 @@ def hasLDU : Predicate<"Subtarget->hasLDU()">;
def hasPTXASUnreachableBug : Predicate<"Subtarget->hasPTXASUnreachableBug()">;
def noPTXASUnreachableBug : Predicate<"!Subtarget->hasPTXASUnreachableBug()">;
def hasOptEnabled : Predicate<"TM.getOptLevel() != CodeGenOptLevel::None">;
-def hasAcceleratedFeatures : Predicate<"Subtarget->hasAAFeatures()">;
+def hasArchAccelFeatures : Predicate<"Subtarget->hasArchAccelFeatures()">;
def doF32FTZ : Predicate<"useF32FTZ()">;
def doNoF32FTZ : Predicate<"!useF32FTZ()">;
diff --git a/llvm/lib/Target/NVPTX/NVPTXIntrinsics.td b/llvm/lib/Target/NVPTX/NVPTXIntrinsics.td
index ed7963f35a7c7..aa80d45d6a5af 100644
--- a/llvm/lib/Target/NVPTX/NVPTXIntrinsics.td
+++ b/llvm/lib/Target/NVPTX/NVPTXIntrinsics.td
@@ -7596,7 +7596,7 @@ multiclass SET_MAXNREG<string Action, Intrinsic Intr> {
def : NVPTXInst<(outs), (ins i32imm:$reg_count),
"setmaxnreg." # Action # ".sync.aligned.u32 $reg_count;",
[(Intr timm:$reg_count)]>,
- Requires<[hasAcceleratedFeatures, hasSM<90>, hasPTX<80>]>;
+ Requires<[hasArchAccelFeatures, hasSM<90>, hasPTX<80>]>;
}
defm INT_SET_MAXNREG_INC : SET_MAXNREG<"inc", int_nvvm_setmaxnreg_inc_sync_aligned_u32>;
diff --git a/llvm/lib/Target/NVPTX/NVPTXSubtarget.h b/llvm/lib/Target/NVPTX/NVPTXSubtarget.h
index 851c9152e4cb8..09d7cb9837a98 100644
--- a/llvm/lib/Target/NVPTX/NVPTXSubtarget.h
+++ b/llvm/lib/Target/NVPTX/NVPTXSubtarget.h
@@ -125,11 +125,11 @@ class NVPTXSubtarget : public NVPTXGenSubtargetInfo {
unsigned int getSmVersion() const { return getFullSmVersion() / 10; }
// GPUs with "a" suffix have include architecture-accelerated features that
// are supported on the specified architecture only, hence such targets do not
- // follow the onion layer model. hasAAFeatures() allows distinguishing such
- // GPU variants from the base GPU architecture.
+ // follow the onion layer model. hasArchAccelFeatures() allows
+ // distinguishing such GPU variants from the base GPU architecture.
// - 0 represents base GPU model,
// - non-zero value identifies particular architecture-accelerated variant.
- bool hasAAFeatures() const { return getFullSmVersion() % 10; }
+ bool hasArchAccelFeatures() const { return getFullSmVersion() % 10; }
// If the user did not provide a target we default to the `sm_30` target.
std::string getTargetName() const {
``````````
</details>
https://github.com/llvm/llvm-project/pull/127990
More information about the llvm-commits
mailing list