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

[Douglas Gregor]

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