[PATCH] D31042: Allow DataLayout to specify addrspace for allocas.

John McCall via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 27 15:18:54 PDT 2017


rjmccall added a comment.

In https://reviews.llvm.org/D31042#711648, @arsenm wrote:

> LLVM treats 0 as the "generic" address space. This is how the LangRef refers to it.


You're revising LangRef here one way or the other.  I'm suggesting that a better revision would be to stop saying that address space 0 is the "generic" address space (because there is no such thing), and instead say that it is the address space of the stack.  Is there anything actually preventing you from doing that?


https://reviews.llvm.org/D31042





More information about the llvm-commits mailing list