[cfe-commits] [Review] Adding a sig_atomic_t type to TargetInfo

Edward O'Callaghan eocallaghan at auroraux.org
Fri Nov 20 16:50:29 PST 2009


Looks nice Ken,

Committed in revision 89520.

Many thanks,
Edward.

2009/11/20 Douglas Gregor <dgregor at apple.com>:
>
> On Nov 20, 2009, at 6:56 AM, Ken Dyck wrote:
>
>> The enclosed patch adds a new SigAtomicType to TargetInfo with the
>> intention of eventually using it to specify values for SIG_ATOMIC_MAX
>> and SIG_ATOMIC_MIN in clang's stdint.h.
>>
>> Currently, clang's stdint.h assumes that sig_atomic_t is a 32-bit
>> signed
>> integer type on every target. This patch maintains that assumption.
>> The
>> default type is 'signed int', which is 32-bits wide on all the
>> supported
>> architectures except MSP430 and PIC16. On these, the default is
>> overridden with their 32-bit 'signed long int' type.
>
> Looks reasonable to me...
>
>        - Doug
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>



-- 
--
Edward O'Callaghan
http://www.auroraux.org/
eocallaghan at auroraux dot org
---
()  ascii ribbon campaign - against html e-mail
/\                        - against microsoft attachments




More information about the cfe-commits mailing list