[llvm-commits] Patch to add address space support to getIntPtrTy

Villmow, Micah Micah.Villmow at amd.com
Thu Oct 18 15:12:16 PDT 2012


I've attached a new patch, and I've also added the test case which fails without this change. There were a few places where I didn't know what to change and my original guess was wrong, so I added FIXME's to those locations. Another thing I think will make these patches better is by adding a new interface of getIntPtrTy(). Currently it is getIntPtrTy(LLVMContext, unsigned) and usually the context and the AS come from the same Value*, so I'm wondering if adding getIntPtrTy(Value*) would be useful.

What do you think?
Thanks again,
Micah
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 002-per-address-space-get-int-ptr-type-patch.txt
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121018/a25ea4c5/attachment.txt>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: 003-per-address-space-test-case.txt
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121018/a25ea4c5/attachment-0001.txt>


More information about the llvm-commits mailing list