[llvm-branch-commits] [llvm] [Attributor] Use `getAssumedAddrSpace` to get address space for `AllocaInst` (PR #136865)

[Shilei Tian via llvm-branch-commits]

shiltian wrote:

> This looks like an attempt to fix up broken IR producers, but I guess that's not it?

Yeah, I initially thought that was broken IR too. At first, I was in favor of not allowing alloca in AS0 at all and just making it a verifier error, like what was done in https://github.com/llvm/llvm-project/commit/c9c1eefa7714fccc3661d79e690fc17945ab7fe1. But @arsenm convinced me otherwise in https://github.com/llvm/llvm-project/pull/135820#issuecomment-2818230936, where the idea is to relax the restriction and fix things up later in the backend.

https://github.com/llvm/llvm-project/pull/136865