[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