OpenCL address space and mangling
david.tweed at arm.com
Wed Sep 11 02:55:29 PDT 2013
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.
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.
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
> for me, because I haven't commit access.
> Thanks in advance.
> Best Regards,
>> - 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
>>>> to this solution.
>>>> The mangler now has a bug, so it must be fixed. The pure solution
>>>> breaks the binary compatibility. If we do not have problem with this
>>>> consider a matter for the users to solve the problem, e.g. with a
>>>> of libraries) the right patch is to have a target independent mangling
>>>> Still we would have problems if we consider SPIR: in its specification
>>>> a fixed mangling scheme (that it's the one produced by the current
>>>> this case we have two choice: we change the SPIR mangling or we allow
>>>> override the target independent mangling for OpenCL with the one based
>>>> *Based on all this would see the mangling proposed by Eli the default
>>>> targets that explicitly requires a mangling scheme based on the target
>>>> spaces map (e.g. the SPIR target).*
>>> In attachment a proposal to implement target independent mangling with
>>> option for targets to force the use of target address space based
>>> cfe-commits mailing list
>>> cfe-commits at cs.uiuc.edu
More information about the cfe-commits