This patch fixes atomic.c for big endian configurations.

diff --git a/libgcc/ChangeLog b/libgcc/ChangeLog
index 56c58dd..bcd6cb1 100644
--- a/libgcc/ChangeLog
+++ b/libgcc/ChangeLog
@@ -1,5 +1,11 @@
 2014-01-25  Walter Lee  <w...@tilera.com>
 
+       * config/tilepro/atomic.c (BIT_OFFSET): Define.
+       (__atomic_subword_cmpxchg): Use BIT_OFFSET.
+       (__atomic_subword): Ditto.
+
+2014-01-25  Walter Lee  <w...@tilera.com>
+
        * config/tilepro/atomic.c (__atomic_do_and_fetch): Add
        a prefix op argument.
        (__atomic_nand_fetch_4): Add prefix op.
diff --git a/libgcc/config/tilepro/atomic.c b/libgcc/config/tilepro/atomic.c
index 9919323..2ad114a 100644
--- a/libgcc/config/tilepro/atomic.c
+++ b/libgcc/config/tilepro/atomic.c
@@ -135,6 +135,12 @@ __atomic_exchange_##size(volatile type* ptr, type val, int 
model)  \
 __atomic_exchange_methods (int, 4)
 __atomic_exchange_methods (long long, 8)
 
+#ifdef __LITTLE_ENDIAN__
+#define BIT_OFFSET(n, type) ((n) * 8)
+#else
+#define BIT_OFFSET(n, type) ((4 - sizeof(type) - (n)) * 8)
+#endif
+
 /* Subword methods require the same approach for both TILEPro and
    TILE-Gx.  We load the background data for the word, insert the
    desired subword piece, then compare-and-exchange it into place.  */
@@ -150,7 +156,7 @@ __atomic_compare_exchange_##size(volatile type* ptr, type* 
guess,   \
 {                                                                      \
   pre_atomic_barrier(models);                                          \
   unsigned int *p = (unsigned int *)((unsigned long)ptr & ~3UL);       \
-  const int shift = ((unsigned long)ptr & 3UL) * 8;                    \
+  const int shift = BIT_OFFSET((unsigned long)ptr & 3UL, type);                
\
   const unsigned int valmask = (1 << (sizeof(type) * 8)) - 1;          \
   const unsigned int bgmask = ~(valmask << shift);                     \
   unsigned int oldword = *p;                                           \
@@ -177,7 +183,7 @@ proto                                                       
                \
 {                                                                      \
   top                                                                  \
   unsigned int *p = (unsigned int *)((unsigned long)ptr & ~3UL);       \
-  const int shift = ((unsigned long)ptr & 3UL) * 8;                    \
+  const int shift = BIT_OFFSET((unsigned long)ptr & 3UL, type);                
\
   const unsigned int valmask = (1 << (sizeof(type) * 8)) - 1;          \
   const unsigned int bgmask = ~(valmask << shift);                     \
   unsigned int oldword, xword = *p;                                    \

Reply via email to