[PATCH] Change objectsize intrinsic to accept different address spaces.

Matt Arsenault arsenm2 at gmail.com
Thu Oct 3 17:54:08 PDT 2013

On Oct 3, 2013, at 17:42 , Eric Christopher <echristo at gmail.com> wrote:

> Oh sad.
> Can you explain a bit more real fast? Thanks!
> -eric
> On Thu, Oct 3, 2013 at 5:17 PM, Matt Arsenault
> <Matthew.Arsenault at amd.com> wrote:
>> Hi nlopes,
>> Bitcasting everything to i8* won't work. Autoupgrade the old
>> intrinsic declarations to use the new mangling.

Pointers to different address spaces aren't necessarily the same size, and soon bitcasts between them won't be allowed at all in favor of adding the addrspace cast instruction. On some targets, some casts between address spaces don't make any sense

More information about the llvm-commits mailing list