Re: [PATCH] Power/GCC: Subword atomic operation endianness check bug fix

2014-07-02 Thread Maciej W. Rozycki
On Wed, 2 Jul 2014, David Edelsohn wrote: > * config/rs6000/rs6000.c (rs6000_adjust_atomic_subword): Use > BYTES_BIG_ENDIAN rather than WORDS_BIG_ENDIAN to check for byte > endianness. > > This patch is okay. Thanks for noticing it. Committed, thanks. Maciej

Re: [PATCH] Power/GCC: Subword atomic operation endianness check bug fix

2014-07-02 Thread David Edelsohn
gcc/ * config/rs6000/rs6000.c (rs6000_adjust_atomic_subword): Use BYTES_BIG_ENDIAN rather than WORDS_BIG_ENDIAN to check for byte endianness. This patch is okay. Thanks for noticing it. Thanks, David

Re: [PATCH] Power/GCC: Subword atomic operation endianness check bug fix

2014-07-02 Thread Alan Modra
On Wed, Jul 02, 2014 at 11:05:23AM +0100, Maciej W. Rozycki wrote: > As it > happens both macros have the same value for the Power target Thanks! Looks good to me. As you note above it is effectively a documentation fix. > --- gcc-fsf-trunk-quilt.orig/gcc/config/rs6000/rs6000.c 2014-06-1