-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 12/06/11 05:21, Jakub Jelinek wrote:
> Hi!
> 
> IMNSHO we want .cfi_restore rbp on leave insn even when that insn
> isn't setting cfa (on the shrink-wrapped testcase below already the
> previous load of %r10 sets cfa, but there is no .cfi_restore rbp
> and we ICE when processing the merge label to which pre-prologue bb
> jumps (which obviously doesn't have %rbp saved or restored).  When
> not emitting leave, but instead doing
> ix86_emit_restore_reg_using_pop, we emit the restore note too.
> 
> Fixed thusly, bootstrapped/regtested on x86_64-linux and
> i686-linux, ok for trunk?
> 
> 2011-12-06  Jakub Jelinek  <ja...@redhat.com>
> 
> PR target/51354 * config/i386/i386.c (ix86_emit_leave): Call
> ix86_add_cfa_restore_note unconditionally.
> 
> * gcc.c-torture/compile/pr51354.c: New test.
OK.
jeff
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQEcBAEBAgAGBQJO3mBCAAoJEBRtltQi2kC7w/AIAKapa2Xlj+rdFVZkRysnjpqJ
kpBVle3paWt9JN8YTw3fPf6WffLqWT3iRcJ51YT7bMd0YVehfCJCUiMY4eRc7C2t
di4ARhXuPpIzuQ5TnvZK5ZpAr6bI8OmwFQHpgLcUxPC0QPJGE5BOpQ3PnW58TzZ0
6p0WyAaLY5AfnuTvbBqajUt7MljLqEv9Ev5jWzmSAZjueb3ltFGPg0IESz0f/F3e
r4lqAsz9RXHsG20ILfFG9aruGIXpytYeAhMdtp3KgiOfN/Tk/BsHNTgkTI7BzMFY
VqA80J6h47qFFmYeBWSgn/YJ2OtUigoiR6QwU+tiLcIh1AKirzMLnMQwwP6eNxo=
=nEJo
-----END PGP SIGNATURE-----

Reply via email to