Message ID | 5613B0B2.6070003@foss.arm.com |
---|---|
State | New |
Headers | show |
On 6 October 2015 at 12:29, Ramana Radhakrishnan <ramana.radhakrishnan@foss.arm.com> wrote: > Thanks for the explanation Eric, by that explanation I do not see the need to adjust for TARGET_EXPR or mark_addressable in the backends. > > Here are the patches that I'm testing - I will apply the ARM one after testing finishes - my previous testing broke because of some other reasons. > > The AArch64 patch cleared testing - ok to apply ? > > PR c/65345 > > * config/aarch64/aarch64-builtins.c (aarch64_atomic_assign_expand_fenv): Use create_tmp_var_raw instead of create_tmp_var. OK /Marcus
diff --git a/gcc/config/arm/arm-builtins.c b/gcc/config/arm/arm-builtins.c index 0f5a1f1..17cf8e7 100644 --- a/gcc/config/arm/arm-builtins.c +++ b/gcc/config/arm/arm-builtins.c @@ -2974,7 +2974,7 @@ arm_atomic_assign_expand_fenv (tree *hold, tree *clear, tree *update) __builtin_arm_set_fpscr (masked_fenv); */ - fenv_var = create_tmp_var (unsigned_type_node); + fenv_var = create_tmp_var_raw (unsigned_type_node); get_fpscr = arm_builtin_decls[ARM_BUILTIN_GET_FPSCR]; set_fpscr = arm_builtin_decls[ARM_BUILTIN_SET_FPSCR]; mask = build_int_cst (unsigned_type_node, @@ -3001,7 +3001,7 @@ arm_atomic_assign_expand_fenv (tree *hold, tree *clear, tree *update) __atomic_feraiseexcept (new_fenv_var); */ - new_fenv_var = create_tmp_var (unsigned_type_node); + new_fenv_var = create_tmp_var_raw (unsigned_type_node); reload_fenv = build2 (MODIFY_EXPR, unsigned_type_node, new_fenv_var, build_call_expr (get_fpscr, 0)); restore_fnenv = build_call_expr (set_fpscr, 1, fenv_var);