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

[Eli Friedman]

On Thu, Oct 18, 2012 at 3:12 PM, Villmow, Micah <Micah.Villmow at amd.com> wrote:
> 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.

-
+

Please don't send patches with changes like this.

-Eli