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

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 23 11:22:59 PDT 2017


arsenm added inline comments.


================
Comment at: docs/LangRef.rst:1816
+``A<address space``
+    Specifies the address space of the result for the stack. Defaults to the
+    default address space of 0.
----------------
efriedma wrote:
> "the address space of the result for the stack" is a little unclear.
I wasn't sure about referring to the stack at all. Should I rename everything added that says stack with alloca (e.g DL.getStackAddrSpace() -> DL.getAllocaAddrSpace()?)


https://reviews.llvm.org/D31042





More information about the llvm-commits mailing list