[llvm] a8fb8eb - AMDGPU: Stop copying triple into AMDGPUSubtarget (#184147)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 2 07:25:44 PST 2026
Author: Matt Arsenault
Date: 2026-03-02T15:25:38Z
New Revision: a8fb8eb49f002abdb05013fcc0072896a7969b50
URL: https://github.com/llvm/llvm-project/commit/a8fb8eb49f002abdb05013fcc0072896a7969b50
DIFF: https://github.com/llvm/llvm-project/commit/a8fb8eb49f002abdb05013fcc0072896a7969b50.diff
LOG: AMDGPU: Stop copying triple into AMDGPUSubtarget (#184147)
Really the triple doesn't belong here at all.
Added:
Modified:
llvm/lib/Target/AMDGPU/AMDGPUSubtarget.h
Removed:
################################################################################
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUSubtarget.h b/llvm/lib/Target/AMDGPU/AMDGPUSubtarget.h
index 302fe7c850b75..4e1e54da5f4a9 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUSubtarget.h
+++ b/llvm/lib/Target/AMDGPU/AMDGPUSubtarget.h
@@ -46,7 +46,7 @@ class AMDGPUSubtarget {
};
private:
- Triple TargetTriple;
+ const Triple &TargetTriple;
protected:
bool HasMulI24 = true;
@@ -61,7 +61,7 @@ class AMDGPUSubtarget {
char WavefrontSizeLog2 = 0;
public:
- AMDGPUSubtarget(Triple TT) : TargetTriple(std::move(TT)) {}
+ AMDGPUSubtarget(const Triple &TT) : TargetTriple(TT) {}
static const AMDGPUSubtarget &get(const MachineFunction &MF);
static const AMDGPUSubtarget &get(const TargetMachine &TM,
More information about the llvm-commits
mailing list