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
_______________________________________________
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to