[llvm] [AMDGPU][Verifier] Check address space of `alloca` instruction (PR #135820)
Shilei Tian via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 24 22:18:33 PDT 2025
================
@@ -4392,6 +4392,12 @@ void Verifier::visitAllocaInst(AllocaInst &AI) {
verifySwiftErrorValue(&AI);
}
+ if (TT.isAMDGPU()) {
----------------
shiltian wrote:
It is because of this folded comment https://github.com/llvm/llvm-project/pull/135820#discussion_r2045137767. There are some test cases that basically have `alloca` in all ASs. Also, based on the discussion in https://github.com/llvm/llvm-project/pull/136865, specifically https://github.com/llvm/llvm-project/pull/136865#issuecomment-2824626809 and https://github.com/llvm/llvm-project/pull/136865#issuecomment-2824517958, we can't rely on DL for this purpose.
https://github.com/llvm/llvm-project/pull/135820
More information about the llvm-commits
mailing list