https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104274

David Malcolm <dmalcolm at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Last reconfirmed|                            |2022-02-10
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |ASSIGNED

--- Comment #1 from David Malcolm <dmalcolm at gcc dot gnu.org> ---
Confirmed (with --target=hppa64-hpux11.3)

pr97029.c.006t.gimple with x86_64 has...

VVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVV    
void setjmp (struct vj pl)
{
  setjmp (pl);
}
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^


...whereas with  --target=hppa64-hpux11.3 has:
VVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVVV    
void setjmp (struct vj pl)
{
  struct vj pl.0;

  setjmp (pl.0);
}
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

It still happens if I rename the function from "setjmp" to "foo".
It no longer happens if I add a dummy field to the struct.

The temporary "pl.0" is created on hppa by:
#4  0x0000000000aa7b3a in gimplify_parameters
(cleanup=cleanup@entry=0x7fffffffd9b0) at ../../src/gcc/function.cc:3939
3926              tree type = TREE_TYPE (data.arg.type);
3927              function_arg_info orig_arg (type, data.arg.named);
3928              if (reference_callee_copied (&all.args_so_far_v, orig_arg))
3929                {
3930                  tree local, t;
3931    
3932                  /* For constant-sized objects, this is trivial; for
3933                     variable-sized objects, we have to play games.  */
3934                  if (TREE_CODE (DECL_SIZE_UNIT (parm)) == INTEGER_CST
3935                      && !(flag_stack_check == GENERIC_STACK_CHECK
3936                           && compare_tree_int (DECL_SIZE_UNIT (parm),
3937                                                STACK_CHECK_MAX_VAR_SIZE) >
0))
3938                    {
3939                      local = create_tmp_var (type, get_name (parm));
3940                      DECL_IGNORED_P (local) = 0;
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

and never assigned to.

Reply via email to