[PATCH] Change objectsize intrinsic to accept different address spaces.
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
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!
>> 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