diff --git a/gcc/ChangeLog b/gcc/ChangeLog index bbe300f6f60..32a21d28e80 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2012-01-27 Richard Henderson + + * config/sparc/sparc.c (TARGET_ATOMIC_TEST_AND_SET_TRUEVAL): New. + * config/sparc/sync.md (atomic_test_and_set): Only handle QImode. + (ldstub): Rename from ldstubqi. + (ldstub): Remove. + 2012-01-27 Richard Henderson * target.def (TARGET_ATOMIC_TEST_AND_SET_TRUEVAL): New. diff --git a/gcc/config/sparc/sparc.c b/gcc/config/sparc/sparc.c index 19ab54a1304..1b3b4c8764c 100644 --- a/gcc/config/sparc/sparc.c +++ b/gcc/config/sparc/sparc.c @@ -779,6 +779,10 @@ char sparc_hard_reg_printed[8]; #undef TARGET_PRINT_OPERAND_ADDRESS #define TARGET_PRINT_OPERAND_ADDRESS sparc_print_operand_address +/* The value stored by LDSTUB. */ +#undef TARGET_ATOMIC_TEST_AND_SET_TRUEVAL +#define TARGET_ATOMIC_TEST_AND_SET_TRUEVAL 0xff + struct gcc_target targetm = TARGET_INITIALIZER; static void diff --git a/gcc/config/sparc/sync.md b/gcc/config/sparc/sync.md index be8c4c4783a..d07d572c614 100644 --- a/gcc/config/sparc/sync.md +++ b/gcc/config/sparc/sync.md @@ -242,25 +242,30 @@ "swap\t%1, %0" [(set_attr "type" "multi")]) -(define_expand "atomic_test_and_set" - [(match_operand:I124MODE 0 "register_operand" "") - (match_operand:I124MODE 1 "memory_operand" "") +(define_expand "atomic_test_and_set" + [(match_operand:QI 0 "register_operand" "") + (match_operand:QI 1 "memory_operand" "") (match_operand:SI 2 "const_int_operand" "")] "" { enum memmodel model = (enum memmodel) INTVAL (operands[2]); + rtx ret; sparc_emit_membar_for_model (model, 3, 1); - - if (mode != QImode) - operands[1] = adjust_address (operands[1], QImode, 0); - emit_insn (gen_ldstub (operands[0], operands[1])); - + emit_insn (gen_ldstub (operands[0], operands[1])); sparc_emit_membar_for_model (model, 3, 2); + + /* Convert the 0/0xff result we would otherwise have to a boolean. + I.e. ignore all but bit 0. */ + ret = expand_simple_binop (QImode, AND, operands[0], const1_rtx, + operands[0], true, OPTAB_LIB_WIDEN); + if (ret != operands[0]) + emit_move_insn (operands[0], ret); + DONE; }) -(define_insn "ldstubqi" +(define_insn "ldstub" [(set (match_operand:QI 0 "register_operand" "=r") (unspec_volatile:QI [(match_operand:QI 1 "memory_operand" "+m")] UNSPECV_LDSTUB)) @@ -268,13 +273,3 @@ "" "ldstub\t%1, %0" [(set_attr "type" "multi")]) - -(define_insn "ldstub" - [(set (match_operand:I24MODE 0 "register_operand" "=r") - (zero_extend:I24MODE - (unspec_volatile:QI [(match_operand:QI 1 "memory_operand" "+m")] - UNSPECV_LDSTUB))) - (set (match_dup 1) (const_int -1))] - "" - "ldstub\t%1, %0" - [(set_attr "type" "multi")])