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

Eric Christopher echristo at gmail.com
Thu Oct 3 18:03:25 PDT 2013

I'm curious how you're generating the intrinsics that would need the
addrspace cast? I'm not necessarily objecting, just figuring out if
it's a change that necessarily needs to be made or if a change in use
would suffice.



On Thu, Oct 3, 2013 at 5:54 PM, Matt Arsenault <arsenm2 at gmail.com> wrote:
> 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