[PATCH] D22488: [IR] Introduce a non-integral pointer type

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 28 11:49:16 PDT 2016


arsenm added inline comments.

================
Comment at: include/llvm/IR/DataLayout.h:206
@@ -201,2 +205,3 @@
     Pointers = DL.Pointers;
+    NonIntegralAddressSpace = DL.NonIntegralAddressSpace;
     return *this;
----------------
arsenm wrote:
> Only one non-integral is supported? I would expect it to support specifying any address space (like the pointer bit widths). I'm not sure we really need more than one, but 2 or 3 seems likely.
If the one restriction remains it at least needs a mention in the langref 


https://reviews.llvm.org/D22488





More information about the llvm-commits mailing list