From: John David Anglin Date: Sat, 29 Oct 2016 15:30:26 +0000 (+0000) Subject: pa.h (BIGGEST_ALIGNMENT): Adjust comment. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=2bc99a1a94d85532caa3c20bbb58cea6df2ebd6c;p=gcc.git pa.h (BIGGEST_ALIGNMENT): Adjust comment. * config/pa/pa.h (BIGGEST_ALIGNMENT): Adjust comment. (MALLOC_ABI_ALIGNMENT): Define to 128 on all targets except SOM. Adjust comment. From-SVN: r241676 --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 529b58e50e2..2a74c71ef65 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2016-10-29 John David Anglin + + * config/pa/pa.h (BIGGEST_ALIGNMENT): Adjust comment. + (MALLOC_ABI_ALIGNMENT): Define to 128 on all targets except SOM. + Adjust comment. + 2016-10-28 Jeff Law * config/vax/vax.h (REGNO_REG_CLASS): Access the REGNO argument. diff --git a/gcc/config/pa/pa.h b/gcc/config/pa/pa.h index f0aa1ae68c6..77874821314 100644 --- a/gcc/config/pa/pa.h +++ b/gcc/config/pa/pa.h @@ -295,17 +295,19 @@ typedef struct GTY(()) machine_function /* No data type wants to be aligned rounder than this. The long double type has 16-byte alignment on the 64-bit target even though it was never implemented in hardware. The software implementation only needs 8-byte - alignment. This is to match the HP compilers. */ + alignment. This matches the biggest alignment of the HP compilers. */ #define BIGGEST_ALIGNMENT (2 * BITS_PER_WORD) /* Alignment, in bits, a C conformant malloc implementation has to provide. The HP-UX malloc implementation provides a default alignment of 8 bytes. - This can be increased with mallopt. The glibc implementation also provides - 8-byte alignment. Note that this isn't enough for various POSIX types such - as pthread_mutex_t. However, since we no longer need the 16-byte alignment - for atomic operations, we ignore the nominal alignment specified for these - types. The same is true for long double on 64-bit HP-UX. */ -#define MALLOC_ABI_ALIGNMENT (64) + It should be 16 bytes on the 64-bit target since long double has 16-byte + alignment. It can be increased with mallopt but it's non critical since + long double was never implemented in hardware. The glibc implementation + currently provides 8-byte alignment. It should be 16 bytes since various + POSIX types such as pthread_mutex_t require 16-byte alignment. Again, + this is non critical since 16-byte alignment is no longer needed for + atomic operations. */ +#define MALLOC_ABI_ALIGNMENT (TARGET_SOM ? 64 : 128) /* Get around hp-ux assembler bug, and make strcpy of constants fast. */ #define CONSTANT_ALIGNMENT(EXP, ALIGN) \