diff options
| author | Jason Merrill <jason@redhat.com> | 2019-10-17 15:09:53 -0400 |
|---|---|---|
| committer | Jason Merrill <jason@gcc.gnu.org> | 2019-10-17 15:09:53 -0400 |
| commit | 8e5993e236a50ffc161dd45d1ef3ab86133e3363 (patch) | |
| tree | c5450036b286821b43db98f6d8a071af778b76bf /gcc/targhooks.c | |
| parent | c11cccc0285f02f117a1e80924fb7673b6486ce9 (diff) | |
| download | gcc-8e5993e236a50ffc161dd45d1ef3ab86133e3363.zip gcc-8e5993e236a50ffc161dd45d1ef3ab86133e3363.tar.gz gcc-8e5993e236a50ffc161dd45d1ef3ab86133e3363.tar.bz2 | |
cp-gimplify.c (cp_gimplify_expr): Use get_initialized_tmp_var.
* cp-gimplify.c (cp_gimplify_expr): Use get_initialized_tmp_var.
The comment for get_formal_tmp_var says that it shouldn't be used for
expressions whose value might change between initialization and use, and in
this case we're creating a temporary precisely because the value might
change, so we should use get_initialized_tmp_var instead.
I also noticed that many callers of get_initialized_tmp_var pass NULL for
post_p, so it seems appropriate to make it a default argument.
gcc/
* gimplify.h (get_initialized_tmp_var): Add default argument to
post_p.
* gimplify.c (gimplify_self_mod_expr, gimplify_omp_atomic): Remove
NULL post_p argument.
* targhooks (std_gimplify_va_arg_expr): Likewise.
From-SVN: r277128
Diffstat (limited to 'gcc/targhooks.c')
| -rw-r--r-- | gcc/targhooks.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/gcc/targhooks.c b/gcc/targhooks.c index 5445038..fee4cc2 100644 --- a/gcc/targhooks.c +++ b/gcc/targhooks.c @@ -2164,11 +2164,11 @@ std_gimplify_va_arg_expr (tree valist, tree type, gimple_seq *pre_p, real_part = std_gimplify_va_arg_expr (valist, TREE_TYPE (type), pre_p, NULL); - real_part = get_initialized_tmp_var (real_part, pre_p, NULL); + real_part = get_initialized_tmp_var (real_part, pre_p); imag_part = std_gimplify_va_arg_expr (unshare_expr (valist), TREE_TYPE (type), pre_p, NULL); - imag_part = get_initialized_tmp_var (imag_part, pre_p, NULL); + imag_part = get_initialized_tmp_var (imag_part, pre_p); return build2 (COMPLEX_EXPR, type, real_part, imag_part); } @@ -2186,7 +2186,7 @@ std_gimplify_va_arg_expr (tree valist, tree type, gimple_seq *pre_p, boundary /= BITS_PER_UNIT; /* Hoist the valist value into a temporary for the moment. */ - valist_tmp = get_initialized_tmp_var (valist, pre_p, NULL); + valist_tmp = get_initialized_tmp_var (valist, pre_p); /* va_list pointer is aligned to PARM_BOUNDARY. If argument actually requires greater alignment, we must perform dynamic alignment. */ |