OpenCL address space and mangling

David Tweed david.tweed at arm.com
Wed Sep 11 02:55:29 PDT 2013


Hi,

while I think most people agree with the direction things are going there
look to still
be some fiddly details. As one instance, when I run this on a standard OSS
LLVM build I get
a new test failure in test/CodeGenOpenCL/local.cl. Since behaviour is being
made more sophisticated, it seems
it would be good to have add some tests that verify the new behaviour so we
can detect any
modifications that change it. But the patch looks to be progressing.

Cheers,
Dave

-----Original Message-----
From: Michele Scandale [mailto:michele.scandale at gmail.com] 
Sent: 10 September 2013 20:23
To: Mon Ping Wang
Cc: cfe-dev at cs.uiuc.edu; cfe-commits at cs.uiuc.edu; Tanya Lattner; David
Tweed; Eli Friedman
Subject: Re: OpenCL address space and mangling

Reup and updated version of the patch!

Thanks in advance.

Regards,
-Michele

On 09/06/2013 01:48 PM, Michele Scandale wrote:
> On 09/06/2013 12:54 PM, Mon Ping Wang wrote:
>> This patch looks fine to me.
> 
> If this patch seems generally fine, I would appreciate if someone can
commit it
> for me, because I haven't commit access.
> 
> Thanks in advance.
> 
> Best Regards,
> -Michele
> 
>>   - Mon Ping
>>
>> On Aug 27, 2013, at 4:12 PM, Michele Scandale
<michele.scandale at gmail.com> wrote:
>>
>>> On 08/24/2013 04:02 PM, Michele Scandale wrote:
>>>> The pure solution would be the one proposed by Eli: I don't have any
objection
>>>> to this solution.
>>>> The mangler now has a bug, so it must be fixed. The pure solution
implicitly
>>>> breaks the binary compatibility. If we do not have problem with this
(so we
>>>> consider a matter for the users to solve the problem, e.g. with a
forced update
>>>> of libraries) the right patch is to have a target independent mangling
for OpenCL.
>>>>
>>>> Still we would have problems if we consider SPIR: in its specification
there is
>>>> a fixed mangling scheme (that it's the one produced by the current
mangler). In
>>>> this case we have two choice: we change the SPIR mangling or we allow
targets to
>>>> override the target independent mangling for OpenCL with the one based
on the
>>>> TargetAddrSpaceMap.
>>>>
>>>> *Based on all this would see the mangling proposed by Eli the default
except for
>>>> targets that explicitly requires a mangling scheme based on the target
address
>>>> spaces map (e.g. the SPIR target).*
>>>
>>> In attachment a proposal to implement target independent mangling with
the
>>> option for targets to force the use of target address space based
mangling.
>>>
>>> Regards,
>>> -Michele
>>> <mangling-rev5.patch>_______________________________________________
>>> cfe-commits mailing list
>>> cfe-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>>
> 







More information about the cfe-commits mailing list