[Bug target/67226] Incorrect code generated for tail call, where parameters are structs passed by value, -O2 is used, and target is ARM

[Bug target/67226] Incorrect code generated for tail call, where parameters are structs passed by value, -O2 is used, and target is ARM

[mikpelinux at gmail dot com]

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

Mikael Pettersson <mikpelinux at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mikpelinux at gmail dot com

--- Comment #1 from Mikael Pettersson <mikpelinux at gmail dot com> ---
Created attachment 36189
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=36189&action=edit
reduced test case

I can reproduce the wrong-code on Cortex-A9 with gcc-6-20150809 at -O2/Os/O3,
but not at -O1/O0/Og.

[Bug target/67226] Incorrect code generated for tail call, where parameters are structs passed by value, -O2 is used, and target is ARM

[mikpelinux at gmail dot com]

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

--- Comment #2 from Mikael Pettersson <mikpelinux at gmail dot com> ---
This bug is clearly related to PR65358.  This PR's test case is NOT fixed by
the final fix for PR65358 (r223753), but it IS fixed (on both trunk and gcc-5
branch) by the initial proposed fix for PR65358:
https://gcc.gnu.org/ml/gcc-patches/2015-03/msg01014.html

Thus it seems that the changes made to that fix between the initial submission
and the final commit broke it.

[Bug target/67226] Incorrect code generated for tail call, where parameters are structs passed by value, -O2 is used, and target is ARM

[rguenth at gcc dot gnu.org]

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

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |wrong-code
             Target|                            |arm
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2015-08-17
     Ever confirmed|0                           |1

[Bug target/67226] Incorrect code generated for tail call, where parameters are structs passed by value, -O2 is used, and target is ARM

[ktkachov at gcc dot gnu.org]

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

ktkachov at gcc dot gnu.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Last reconfirmed|2015-08-17 00:00:00         |2015-8-18
                 CC|                            |ktkachov at gcc dot gnu.org

--- Comment #3 from ktkachov at gcc dot gnu.org ---
Confirmed as well.
The cause seems to be that one of the callsites of emit_push_insn within
store_one_arg does not set sibcall_p to true when it should.
I'll test a fix.

[Bug rtl-optimization/67226] Incorrect code generated for tail call, where parameters are structs passed by value, -O2 is used, and target is ARM

[pinskia at gcc dot gnu.org]

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

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |alexandre.nunes at gmail dot com

--- Comment #11 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
*** Bug 63206 has been marked as a duplicate of this bug. ***