[Mesa-dev] [PATCH 10/11] clover: Add function for building a clover::module for non-TGSI targets v2

[Francisco Jerez]

Tom Stellard <thomas.stellard at amd.com> writes:

> On Wed, May 23, 2012 at 02:04:20PM +0200, Francisco Jerez wrote:
>>[...]
>> Do we need this compatibility code?  Don't you have to build a patched
>> version of LLVM from sources for this to work anyway?
>> 
>
> You only need a patch if you want to use the r600 target.  This
> compatibility code is necessary if you want to generate code for other
> targets.  I don't mind dropping it, but it would mean that clover won't
> work with LLVM versions older than 3.1
>
That's fine...  In any case, r600 is the only working LLVM target we're
going to have once these changes are applied.

>>[...]
>> > +            /* XXX: Figure out LLVM->OpenCL address space mappings for each
>> > +             * target.  I think we need to ask clang what these are.  For now,
>> > +             * pretend everything is in the global address space. */
>> 
>> Aren't these supposed to be defined arbitrarily by the standard library
>> using "address_space" attributes?
>> 
>
> The OpenCL address space mappings are one of the attributes of a target
> in Clang, they aren't defined in the standard library.
>

OK, makes sense.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 229 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20120523/1495441a/attachment.pgp>