[llvm] [MC] Fix HIP buildbot errors after 68472a3 (PR #141138)
Jan Patrick Lehr via llvm-commits
llvm-commits at lists.llvm.org
Thu May 22 13:47:17 PDT 2025
https://github.com/jplehr created https://github.com/llvm/llvm-project/pull/141138
This fixes the errors on our HIP-Kokkos buildbot after 68472a39a0fbf38f5da7bb4ebe43e2ca87ff8308.
I do not know if this is the actually right fix, but afaict it restores the behavior for AMDGPU before the mentioned patch.
>From e5f5dcf78f26c414eb83168a62a17cf817e23ef8 Mon Sep 17 00:00:00 2001
From: JP Lehr <JanPatrick.Lehr at amd.com>
Date: Thu, 22 May 2025 15:44:06 -0500
Subject: [PATCH] [MC] Fix HIP buildbot errors after 68472a3
---
.../Target/AMDGPU/MCTargetDesc/AMDGPUAsmBackend.cpp | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/llvm/lib/Target/AMDGPU/MCTargetDesc/AMDGPUAsmBackend.cpp b/llvm/lib/Target/AMDGPU/MCTargetDesc/AMDGPUAsmBackend.cpp
index 19e394e5bc38c..0c98c32c21c2c 100644
--- a/llvm/lib/Target/AMDGPU/MCTargetDesc/AMDGPUAsmBackend.cpp
+++ b/llvm/lib/Target/AMDGPU/MCTargetDesc/AMDGPUAsmBackend.cpp
@@ -18,6 +18,7 @@
#include "llvm/MC/MCFixupKindInfo.h"
#include "llvm/MC/MCObjectWriter.h"
#include "llvm/MC/MCSubtargetInfo.h"
+#include "llvm/MC/MCValue.h"
#include "llvm/MC/TargetRegistry.h"
#include "llvm/Support/EndianStream.h"
#include "llvm/TargetParser/TargetParser.h"
@@ -51,6 +52,8 @@ class AMDGPUAsmBackend : public MCAsmBackend {
std::optional<MCFixupKind> getFixupKind(StringRef Name) const override;
MCFixupKindInfo getFixupKindInfo(MCFixupKind Kind) const override;
+ bool shouldForceRelocation(const MCAssembler &, const MCFixup &,
+ const MCValue &, const MCSubtargetInfo *) override;
};
} //End anonymous namespace
@@ -189,6 +192,13 @@ MCFixupKindInfo AMDGPUAsmBackend::getFixupKindInfo(MCFixupKind Kind) const {
return Infos[Kind - FirstTargetFixupKind];
}
+bool AMDGPUAsmBackend::shouldForceRelocation(const MCAssembler &,
+ const MCFixup &,
+ const MCValue &Target,
+ const MCSubtargetInfo *) {
+ return Target.getSpecifier();
+}
+
unsigned AMDGPUAsmBackend::getMinimumNopSize() const {
return 4;
}
More information about the llvm-commits
mailing list