[PATCH] rs6000: Fix rs6000_atomic_assign_expand_fenv [PR94826]
David Edelsohn
dje.gcc@gmail.com
Wed Apr 29 13:29:53 GMT 2020
On Wed, Apr 29, 2020 at 7:48 AM Jakub Jelinek <jakub@redhat.com> wrote:
>
> Hi!
>
> This is the rs6000 version of the earlier committed x86, aarch64 and arm
> fixes, as create_tmp_var_raw is used because the C FE can call this outside
> of function context, we need to make sure the first references to those
> VAR_DECLs are through a TARGET_EXPR, so that it gets gimple_add_tmp_var
> marked in whatever function it gets expanded in. Without that DECL_CONTEXT
> is NULL and the vars aren't added as local decls of the containing function.
>
> Bootstrapped/regtested on powerpc64{,le}-linux, ok for trunk?
>
> 2020-04-29 Jakub Jelinek <jakub@redhat.com>
>
> PR target/94826
> * config/rs6000/rs6000.c (rs6000_atomic_assign_expand_fenv): Use
> TARGET_EXPR instead of MODIFY_EXPR for first assignment to
> fenv_var, fenv_clear and old_fenv variables. For fenv_addr
> take address of TARGET_EXPR of fenv_var with void_node initializer.
> Formatting fixes.
Okay.
Thanks, David
More information about the Gcc-patches
mailing list