Skip to content

Commit 679d2b2

Browse files
authored
AMDGPU: Fix using IRAttribute with nounwind for AMDGPUNoAGPR (#161954)
Don't think this did anything harmful, but it doesn't make sense to report this as implementing nounwind handling.
1 parent f3703f3 commit 679d2b2

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

llvm/lib/Target/AMDGPU/AMDGPUAttributor.cpp

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1224,12 +1224,9 @@ static bool inlineAsmUsesAGPRs(const InlineAsm *IA) {
12241224
}
12251225

12261226
// TODO: Migrate to range merge of amdgpu-agpr-alloc.
1227-
// FIXME: Why is this using Attribute::NoUnwind?
1228-
struct AAAMDGPUNoAGPR
1229-
: public IRAttribute<Attribute::NoUnwind,
1230-
StateWrapper<BooleanState, AbstractAttribute>,
1231-
AAAMDGPUNoAGPR> {
1232-
AAAMDGPUNoAGPR(const IRPosition &IRP, Attributor &A) : IRAttribute(IRP) {}
1227+
struct AAAMDGPUNoAGPR : public StateWrapper<BooleanState, AbstractAttribute> {
1228+
using Base = StateWrapper<BooleanState, AbstractAttribute>;
1229+
AAAMDGPUNoAGPR(const IRPosition &IRP, Attributor &A) : Base(IRP) {}
12331230

12341231
static AAAMDGPUNoAGPR &createForPosition(const IRPosition &IRP,
12351232
Attributor &A) {

0 commit comments

Comments
 (0)