[llvm] [SPIRV] Fix assert in `getOrCreateBaseRegister` for `i32 -1` (PR #114630)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 1 18:00:26 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-spir-v
Author: Justin Bogner (bogner)
<details>
<summary>Changes</summary>
When trying to create a const inst from a 32 bit signed value, we don't want to sign-extend it to 64 bits, as the resulting value won't actually fit in an `i32` if it was negative.
This fixes crashes in the following two tests after the APInt constructor asserts were enabled in #<!-- -->114539:
```
Failed Tests (2):
LLVM :: CodeGen/SPIRV/transcoding/RelationalOperators.ll
LLVM :: CodeGen/SPIRV/uitofp-with-bool.ll
```
---
Full diff: https://github.com/llvm/llvm-project/pull/114630.diff
1 Files Affected:
- (modified) llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp (+1-1)
``````````diff
diff --git a/llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp b/llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp
index 62bd8d1f9d2433..af0df4d6e5d563 100644
--- a/llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp
+++ b/llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp
@@ -400,7 +400,7 @@ Register SPIRVGlobalRegistry::getOrCreateBaseRegister(
}
assert(Type->getOpcode() == SPIRV::OpTypeInt);
SPIRVType *SpvBaseType = getOrCreateSPIRVIntegerType(BitWidth, I, TII);
- return getOrCreateConstInt(Val->getUniqueInteger().getSExtValue(), I,
+ return getOrCreateConstInt(Val->getUniqueInteger().getZExtValue(), I,
SpvBaseType, TII, ZeroAsNull);
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/114630
More information about the llvm-commits
mailing list