[Bug target/60996] New: Bad code (I.e. needless insns) with option momit-leaf-frame-pointer; side-effect on non-leaf functions

[Bug target/60996] New: Bad code (I.e. needless insns) with option momit-leaf-frame-pointer; side-effect on non-leaf functions

[meisenmann.lba@fh-salzburg.ac.at]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60996

            Bug ID: 60996
           Summary: Bad code (I.e. needless insns) with option
                    momit-leaf-frame-pointer; side-effect on non-leaf
                    functions
           Product: gcc
           Version: 4.8.2
            Status: UNCONFIRMED
          Severity: minor
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: meisenmann.lba@fh-salzburg.ac.at

Using the option '-momit-leaf-frame-pointer' (with -fno-omit-frame-pointer) has
a side-effect on non-leaf functions. The code-snippets afterwards (produced
with an i386-elf cross-compiler for an IA32-target) will show the difference:

A) Without the option momit-leaf-frame-pointer:

Example1:
  pushl %ebp
  movl  %esp, %ebp
  pushl %ebx
  subl  $8, %esp
  movl  (%eax), %ebx
  pushl 12(%ebp)
  pushl 8(%ebp)
  pushl %ecx
  pushl %edx
  pushl %eax
  call  *12(%ebx)
  movl  -4(%ebp), %ebx
  leave
  ret

B) Option '-momit-leaf-frame-pointer' used:

Example1:
  pushl %ebp
  movl  %esp, %ebp
  pushl %ebx
  subl  $8, %esp
  movl  (%eax), %ebx
  pushl 12(%ebp)
  pushl 8(%ebp)
  pushl %ecx
  pushl %edx
  pushl %eax
  call  *12(%ebx)
  addl  $24, %esp
  movl  -4(%ebp), %ebx
  leave
  ret

In this case there'a an additional stack-adjustment (addl $24, %esp) before the
"final" stack-leave instruction. Some other examples with less forwarded
call-arguments have shown a few pop-insn instead of an 'add'.
These additional instruction(s) directly before reverting the stack-frame has
no functional effect, but will consume a few CPU-cycles.

[Bug target/60996] Bad code (I.e. needless insns) with option momit-leaf-frame-pointer; side-effect on non-leaf functions

[pinskia at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
   Last reconfirmed|                            |2021-09-27
             Status|UNCONFIRMED                 |WAITING

--- Comment #1 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
Can you attach the testcase?