OpenCL C "long" should always have 64 bits

Erik Schnetter schnetter at gmail.com
Mon Sep 2 18:03:23 PDT 2013


Please disregard the previous patch -- it accidentally used 32 bits for double.

The correct patch is attached.

-erik

On 2013-09-02, at 13:08 , Erik Schnetter <schnetter at gmail.com> wrote:

> I attach an updated patch that
> 
> (1) also sets sizeof(int) to 32 bits; short and char are hardwired in Clang;
> (2) does not introduce address spaces any more.
> 
> -erik
> 
> On 2013-09-02, at 9:04 , David Tweed <david.tweed at arm.com> wrote:
> 
>> Hi,
>> 
>> | The address space map IMO should not be imposed, because it represents how a
>> | given OpenCL address space should be mapped to target address space that the
>> | target backend is able to handle.
>> 
>> I think that the view on how to represent OpenCL address spaces (front-end vs back-end) has changed since I first posted my patch, and I agree this is now inappropriate to hardwire what's now a backend value for all targets.
> 
> -- 
> Erik Schnetter <schnetter at gmail.com>
> http://www.perimeterinstitute.ca/personal/eschnetter/
> 
> My email is as private as my paper mail. I therefore support encrypting
> and signing email messages. Get my PGP key from http://pgp.mit.edu/.
> <OpenCL-long.diff>

-- 
Erik Schnetter <schnetter at gmail.com>
http://www.perimeterinstitute.ca/personal/eschnetter/

My email is as private as my paper mail. I therefore support encrypting
and signing email messages. Get my PGP key from http://pgp.mit.edu/.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenCL-long.diff
Type: application/octet-stream
Size: 3308 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130902/f515865d/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 203 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130902/f515865d/attachment.sig>


More information about the cfe-commits mailing list