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

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

> Do you actually need this to be printed/parsed in .ll files? It's globally consistent across all alloca instructions, and the data layout is always parsed before anything else in the file. It's not the end of the world, just seems odd to imply that this is actually something that can be different on different instructions.

No, strictly speaking it isn't necessary: every module has a datalayout, so we can compute the type from that.  I requested it to make debugging more straightforward. Without it, there isn't any obvious indication if you dump() an instruction or look at an IR file that the address-space of the result is non-zero.


https://reviews.llvm.org/D31042