[Bug target/57288] New: cfi_restore should precede cfi_def_cfa_offset

[Bug target/57288] New: cfi_restore should precede cfi_def_cfa_offset

[msharov at users dot sourceforge.net]

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

            Bug ID: 57288
           Summary: cfi_restore should precede cfi_def_cfa_offset
           Product: gcc
           Version: 4.8.0
            Status: UNCONFIRMED
          Severity: trivial
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: msharov at users dot sourceforge.net

Created attachment 30122
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=30122&action=edit
The emitted assembly exhibiting the ordering problem

This is on x86_64, compiled with -Os. In the attached assembly, line 89, .L55,
.cfi_restore is emitted for ebx and ebp after .cfi_def_cfa_offset 8 already
invalidated the location where they were stored. cfa_offset should be emitted
after cfi_restores, as it was in the other codepaths like .LEHE0-.L51

[Bug target/57288] cfi_restore should precede cfi_def_cfa_offset

[pinskia at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Target|                            |x86_64-*-linux-gnu
             Status|UNCONFIRMED                 |WAITING
   Last reconfirmed|                            |2013-11-10
     Ever confirmed|0                           |1

--- Comment #1 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
Can you attach the preprocessed source which is used to create this assembly
file?

[Bug target/57288] cfi_restore should precede cfi_def_cfa_offset

[msharov at users dot sourceforge.net]

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

--- Comment #2 from Mike Sharov <msharov at users dot sourceforge.net> ---
(In reply to Andrew Pinski from comment #1)
> Can you attach the preprocessed source which is used to create this assembly
> file?

I'm afraid not. This call has been created by a gigantic collection of
templates, macros, and inline functions, so is too large to attach. Futhermore,
when compiled with the current gcc 4.8.2, the .cfi directives are entirely
different, with no .cfi_restore instructions emitted. If you really can't
figure out what the cause was, I'd have to wait until I see another function
showing the behavior.

[Bug target/57288] cfi_restore should precede cfi_def_cfa_offset

[pinskia at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|WAITING                     |RESOLVED
         Resolution|---                         |INVALID

--- Comment #3 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
So closing as invalid.  Next time you can gzip the preprocessed file or even
place it on some public web page if it becomes too big.