|
 |
85359c |
2016-02-04 Jakub Jelinek <jakub@redhat.com>
|
|
|
85359c |
|
|
|
85359c |
Backported from mainline
|
|
|
85359c |
2016-02-03 Jakub Jelinek <jakub@redhat.com>
|
|
|
85359c |
|
|
|
85359c |
PR target/69644
|
|
|
85359c |
* config/rs6000/rs6000.c (rs6000_expand_atomic_compare_and_swap):
|
|
|
85359c |
Force oldval into register if it does not satisfy reg_or_short_operand
|
|
|
85359c |
predicate. Fix up formatting.
|
|
|
85359c |
|
|
|
85359c |
* gcc.dg/pr69644.c: New test.
|
|
|
85359c |
|
|
|
85359c |
--- gcc/config/rs6000/rs6000.c
|
|
|
85359c |
+++ gcc/config/rs6000/rs6000.c
|
|
|
85359c |
@@ -20263,6 +20263,9 @@ rs6000_expand_atomic_compare_and_swap (rtx operands[])
|
|
|
85359c |
else if (reg_overlap_mentioned_p (retval, oldval))
|
|
|
85359c |
oldval = copy_to_reg (oldval);
|
|
|
85359c |
|
|
|
85359c |
+ if (mode != TImode && !reg_or_short_operand (oldval, mode))
|
|
|
85359c |
+ oldval = copy_to_mode_reg (mode, oldval);
|
|
|
85359c |
+
|
|
|
85359c |
mem = rs6000_pre_atomic_barrier (mem, mod_s);
|
|
|
85359c |
|
|
|
85359c |
label1 = NULL_RTX;
|
|
|
85359c |
@@ -20277,10 +20280,8 @@ rs6000_expand_atomic_compare_and_swap (rtx operands[])
|
|
|
85359c |
|
|
|
85359c |
x = retval;
|
|
|
85359c |
if (mask)
|
|
|
85359c |
- {
|
|
|
85359c |
- x = expand_simple_binop (SImode, AND, retval, mask,
|
|
|
85359c |
- NULL_RTX, 1, OPTAB_LIB_WIDEN);
|
|
|
85359c |
- }
|
|
|
85359c |
+ x = expand_simple_binop (SImode, AND, retval, mask,
|
|
|
85359c |
+ NULL_RTX, 1, OPTAB_LIB_WIDEN);
|
|
|
85359c |
|
|
|
85359c |
cond = gen_reg_rtx (CCmode);
|
|
|
85359c |
/* If we have TImode, synthesize a comparison. */
|
|
|
85359c |
--- /dev/null
|
|
|
85359c |
+++ gcc/testsuite/gcc.dg/pr69644.c
|
|
|
85359c |
@@ -0,0 +1,11 @@
|
|
|
85359c |
+/* PR target/69644 */
|
|
|
85359c |
+/* { dg-do compile } */
|
|
|
85359c |
+
|
|
|
85359c |
+int
|
|
|
85359c |
+main ()
|
|
|
85359c |
+{
|
|
|
85359c |
+ unsigned short x = 0x8000;
|
|
|
85359c |
+ if (!__sync_bool_compare_and_swap (&x, 0x8000, 0) || x)
|
|
|
85359c |
+ __builtin_abort ();
|
|
|
85359c |
+ return 0;
|
|
|
85359c |
+}
|