[lld] [llvm] [lld][ELF] improve AMD error message (PR #135082)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 9 14:10:19 PDT 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-lld
@llvm/pr-subscribers-backend-amdgpu
Author: Maksim Levental (makslevental)
<details>
<summary>Changes</summary>
Improve AMD error message to say it's about `amd-hsa` specifically.
---
Full diff: https://github.com/llvm/llvm-project/pull/135082.diff
2 Files Affected:
- (modified) lld/ELF/Arch/AMDGPU.cpp (+5-5)
- (modified) llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp (+2-1)
``````````diff
diff --git a/lld/ELF/Arch/AMDGPU.cpp b/lld/ELF/Arch/AMDGPU.cpp
index 52fc779855a36..25a05da983bd7 100644
--- a/lld/ELF/Arch/AMDGPU.cpp
+++ b/lld/ELF/Arch/AMDGPU.cpp
@@ -79,8 +79,8 @@ uint32_t AMDGPU::calcEFlagsV4() const {
if (retXnack == EF_AMDGPU_FEATURE_XNACK_UNSUPPORTED_V4 ||
(retXnack != EF_AMDGPU_FEATURE_XNACK_ANY_V4 &&
- (getEFlags(f) & EF_AMDGPU_FEATURE_XNACK_V4)
- != EF_AMDGPU_FEATURE_XNACK_ANY_V4)) {
+ (getEFlags(f) & EF_AMDGPU_FEATURE_XNACK_V4) !=
+ EF_AMDGPU_FEATURE_XNACK_ANY_V4)) {
if (retXnack != (getEFlags(f) & EF_AMDGPU_FEATURE_XNACK_V4)) {
Err(ctx) << "incompatible xnack: " << f;
return 0;
@@ -92,8 +92,8 @@ uint32_t AMDGPU::calcEFlagsV4() const {
if (retSramEcc == EF_AMDGPU_FEATURE_SRAMECC_UNSUPPORTED_V4 ||
(retSramEcc != EF_AMDGPU_FEATURE_SRAMECC_ANY_V4 &&
- (getEFlags(f) & EF_AMDGPU_FEATURE_SRAMECC_V4) !=
- EF_AMDGPU_FEATURE_SRAMECC_ANY_V4)) {
+ (getEFlags(f) & EF_AMDGPU_FEATURE_SRAMECC_V4) !=
+ EF_AMDGPU_FEATURE_SRAMECC_ANY_V4)) {
if (retSramEcc != (getEFlags(f) & EF_AMDGPU_FEATURE_SRAMECC_V4)) {
Err(ctx) << "incompatible sramecc: " << f;
return 0;
@@ -143,7 +143,7 @@ uint32_t AMDGPU::calcEFlags() const {
case ELFABIVERSION_AMDGPU_HSA_V6:
return calcEFlagsV6();
default:
- Err(ctx) << "unknown abi version: " << abiVersion;
+ Err(ctx) << "unknown AMD abi version: " << abiVersion;
return 0;
}
}
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp b/llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp
index bbd262748d680..85ea867700968 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp
@@ -815,7 +815,8 @@ enum IGLPStrategyID : int {
MFMASmallGemmOptID = 0,
MFMASmallGemmSingleWaveOptID = 1,
MFMAExpInterleaveID = 2,
- MFMAExpSimpleInterleaveID = 3
+ MFMAExpSimpleInterleaveID = 3,
+ MFMAExpSimpleInterleaveIDMYSCHEDULER = 4
};
// Implement a IGLP scheduling strategy.
``````````
</details>
https://github.com/llvm/llvm-project/pull/135082
More information about the llvm-commits
mailing list