[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