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

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

2014-07-02 Thread Maciej W. Rozycki
Hi, This change: r199935 | amodra | 2013-06-11 07:17:50 +0100 (Tue, 11 Jun 2013) | 4 lines * config/rs6000/rs6000.c (rs6000_adjust_atomic_subword): Calculate correct shift value in little-endian mode.