[all-commits] [llvm/llvm-project] 3bc125: [AMDGPU][Verifier] Check address space of `alloca`...

Shilei Tian via All-commits all-commits at lists.llvm.org
Fri Apr 25 21:54:21 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3bc125490aee0bd4ce508e76265839c798fdc398
      https://github.com/llvm/llvm-project/commit/3bc125490aee0bd4ce508e76265839c798fdc398
  Author: Shilei Tian <i at tianshilei.me>
  Date:   2025-04-26 (Sat, 26 Apr 2025)

  Changed paths:
    M llvm/lib/IR/Verifier.cpp
    R llvm/test/CodeGen/AMDGPU/assert-wrong-alloca-addrspace.ll
    M llvm/test/CodeGen/AMDGPU/lower-indirect-lds-references.ll
    M llvm/test/Transforms/InstCombine/alloca-in-non-alloca-as.ll
    M llvm/test/Transforms/OpenMP/custom_state_machines.ll
    M llvm/test/Transforms/OpenMP/custom_state_machines_pre_lto.ll
    M llvm/test/Transforms/OpenMP/spmdization.ll
    M llvm/test/Transforms/OpenMP/spmdization_constant_prop.ll
    M llvm/test/Transforms/OpenMP/spmdization_indirect.ll
    A llvm/test/Verifier/AMDGPU/alloca.ll

  Log Message:
  -----------
  [AMDGPU][Verifier] Check address space of `alloca` instruction (#135820)

This PR updates the `Verifier` to enforce that `alloca` instructions on
AMDGPU must be in AS5. This prevents hitting a misleading backend error
like "unable to select FrameIndex," which makes it look like a backend
bug when it's actually an IR-level issue.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list