Skip to content

Commit 8a53b25

Browse files
committed
[AMDGPU] Use default member initializers in Subtarget classes
Use default member initializers in AMDGPUSubtarget and subclasses. This is to guard against adding a new feature boolean in AMDGPUSubtarget.h but forgetting to initialize it to false in AMDGPUSubtarget.cpp. This was mostly autogenerated by: clang-tidy -checks=-*,cppcoreguidelines-prefer-member-initializer,modernize-use-default-member-init -header-filter=Subtarget -fix lib/Target/AMDGPU/*Subtarget.cpp Differential Revision: https://reviews.llvm.org/D123613
1 parent 2ac876c commit 8a53b25

File tree

5 files changed

+137
-275
lines changed

5 files changed

+137
-275
lines changed

llvm/lib/Target/AMDGPU/AMDGPUSubtarget.cpp

Lines changed: 1 addition & 137 deletions
Original file line numberDiff line numberDiff line change
@@ -154,26 +154,7 @@ GCNSubtarget::initializeSubtargetDependencies(const Triple &TT,
154154
return *this;
155155
}
156156

157-
AMDGPUSubtarget::AMDGPUSubtarget(const Triple &TT) :
158-
TargetTriple(TT),
159-
GCN3Encoding(false),
160-
Has16BitInsts(false),
161-
HasMadMixInsts(false),
162-
HasMadMacF32Insts(false),
163-
HasDsSrc2Insts(false),
164-
HasSDWA(false),
165-
HasVOP3PInsts(false),
166-
HasMulI24(true),
167-
HasMulU24(true),
168-
HasSMulHi(false),
169-
HasInv2PiInlineImm(false),
170-
HasFminFmaxLegacy(true),
171-
EnablePromoteAlloca(false),
172-
HasTrigReducedRange(false),
173-
MaxWavesPerEU(10),
174-
LocalMemorySize(0),
175-
WavefrontSizeLog2(0)
176-
{ }
157+
AMDGPUSubtarget::AMDGPUSubtarget(const Triple &TT) : TargetTriple(TT) {}
177158

178159
GCNSubtarget::GCNSubtarget(const Triple &TT, StringRef GPU, StringRef FS,
179160
const GCNTargetMachine &TM)
@@ -182,124 +163,7 @@ GCNSubtarget::GCNSubtarget(const Triple &TT, StringRef GPU, StringRef FS,
182163
AMDGPUSubtarget(TT),
183164
TargetTriple(TT),
184165
TargetID(*this),
185-
Gen(INVALID),
186166
InstrItins(getInstrItineraryForCPU(GPU)),
187-
LDSBankCount(0),
188-
MaxPrivateElementSize(0),
189-
190-
FastFMAF32(false),
191-
FastDenormalF32(false),
192-
HalfRate64Ops(false),
193-
FullRate64Ops(false),
194-
195-
FlatForGlobal(false),
196-
AutoWaitcntBeforeBarrier(false),
197-
BackOffBarrier(false),
198-
UnalignedScratchAccess(false),
199-
UnalignedAccessMode(false),
200-
201-
HasApertureRegs(false),
202-
SupportsXNACK(false),
203-
EnableXNACK(false),
204-
EnableTgSplit(false),
205-
EnableCuMode(false),
206-
TrapHandler(false),
207-
208-
EnableLoadStoreOpt(false),
209-
EnableUnsafeDSOffsetFolding(false),
210-
EnableSIScheduler(false),
211-
EnableDS128(false),
212-
EnablePRTStrictNull(false),
213-
DumpCode(false),
214-
215-
FP64(false),
216-
CIInsts(false),
217-
GFX8Insts(false),
218-
GFX9Insts(false),
219-
GFX90AInsts(false),
220-
GFX940Insts(false),
221-
GFX10Insts(false),
222-
GFX10_3Insts(false),
223-
GFX7GFX8GFX9Insts(false),
224-
SGPRInitBug(false),
225-
NegativeScratchOffsetBug(false),
226-
NegativeUnalignedScratchOffsetBug(false),
227-
HasSMemRealTime(false),
228-
HasIntClamp(false),
229-
HasFmaMixInsts(false),
230-
HasMovrel(false),
231-
HasVGPRIndexMode(false),
232-
HasScalarStores(false),
233-
HasScalarAtomics(false),
234-
HasSDWAOmod(false),
235-
HasSDWAScalar(false),
236-
HasSDWASdst(false),
237-
HasSDWAMac(false),
238-
HasSDWAOutModsVOPC(false),
239-
HasDPP(false),
240-
HasDPP8(false),
241-
Has64BitDPP(false),
242-
HasPackedFP32Ops(false),
243-
HasImageInsts(false),
244-
HasExtendedImageInsts(false),
245-
HasR128A16(false),
246-
HasGFX10A16(false),
247-
HasG16(false),
248-
HasNSAEncoding(false),
249-
NSAMaxSize(0),
250-
GFX10_AEncoding(false),
251-
GFX10_BEncoding(false),
252-
HasDLInsts(false),
253-
HasDot1Insts(false),
254-
HasDot2Insts(false),
255-
HasDot3Insts(false),
256-
HasDot4Insts(false),
257-
HasDot5Insts(false),
258-
HasDot6Insts(false),
259-
HasDot7Insts(false),
260-
HasMAIInsts(false),
261-
HasPkFmacF16Inst(false),
262-
HasAtomicFaddInsts(false),
263-
SupportsSRAMECC(false),
264-
EnableSRAMECC(false),
265-
HasNoSdstCMPX(false),
266-
HasVscnt(false),
267-
HasGetWaveIdInst(false),
268-
HasSMemTimeInst(false),
269-
HasShaderCyclesRegister(false),
270-
HasVOP3Literal(false),
271-
HasNoDataDepHazard(false),
272-
FlatAddressSpace(false),
273-
FlatInstOffsets(false),
274-
FlatGlobalInsts(false),
275-
FlatScratchInsts(false),
276-
ScalarFlatScratchInsts(false),
277-
HasArchitectedFlatScratch(false),
278-
EnableFlatScratch(false),
279-
AddNoCarryInsts(false),
280-
HasUnpackedD16VMem(false),
281-
LDSMisalignedBug(false),
282-
HasMFMAInlineLiteralBug(false),
283-
UnalignedBufferAccess(false),
284-
UnalignedDSAccess(false),
285-
HasPackedTID(false),
286-
287-
ScalarizeGlobal(false),
288-
289-
HasVcmpxPermlaneHazard(false),
290-
HasVMEMtoScalarWriteHazard(false),
291-
HasSMEMtoVectorWriteHazard(false),
292-
HasInstFwdPrefetchBug(false),
293-
HasVcmpxExecWARHazard(false),
294-
HasLdsBranchVmemWARHazard(false),
295-
HasNSAtoVMEMBug(false),
296-
HasNSAClauseBug(false),
297-
HasOffset3fBug(false),
298-
HasFlatSegmentOffsetBug(false),
299-
HasImageStoreD16Bug(false),
300-
HasImageGather4D16Bug(false),
301-
302-
FeatureDisable(false),
303167
InstrInfo(initializeSubtargetDependencies(TT, GPU, FS)),
304168
TLInfo(TM, *this),
305169
FrameLowering(TargetFrameLowering::StackGrowsUp, getStackAlignment(), 0) {

llvm/lib/Target/AMDGPU/AMDGPUSubtarget.h

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -45,23 +45,23 @@ class AMDGPUSubtarget {
4545
Triple TargetTriple;
4646

4747
protected:
48-
bool GCN3Encoding;
49-
bool Has16BitInsts;
50-
bool HasMadMixInsts;
51-
bool HasMadMacF32Insts;
52-
bool HasDsSrc2Insts;
53-
bool HasSDWA;
54-
bool HasVOP3PInsts;
55-
bool HasMulI24;
56-
bool HasMulU24;
57-
bool HasSMulHi;
58-
bool HasInv2PiInlineImm;
59-
bool HasFminFmaxLegacy;
60-
bool EnablePromoteAlloca;
61-
bool HasTrigReducedRange;
62-
unsigned MaxWavesPerEU;
63-
unsigned LocalMemorySize;
64-
char WavefrontSizeLog2;
48+
bool GCN3Encoding = false;
49+
bool Has16BitInsts = false;
50+
bool HasMadMixInsts = false;
51+
bool HasMadMacF32Insts = false;
52+
bool HasDsSrc2Insts = false;
53+
bool HasSDWA = false;
54+
bool HasVOP3PInsts = false;
55+
bool HasMulI24 = true;
56+
bool HasMulU24 = true;
57+
bool HasSMulHi = false;
58+
bool HasInv2PiInlineImm = false;
59+
bool HasFminFmaxLegacy = true;
60+
bool EnablePromoteAlloca = false;
61+
bool HasTrigReducedRange = false;
62+
unsigned MaxWavesPerEU = 10;
63+
unsigned LocalMemorySize = 0;
64+
char WavefrontSizeLog2 = 0;
6565

6666
public:
6767
AMDGPUSubtarget(const Triple &TT);

0 commit comments

Comments
 (0)