https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
ktkachov at gcc dot gnu.org changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Known to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
--- Comment #14 from sudi at gcc dot gnu.org ---
Author: sudi
Date: Thu Mar 29 09:27:53 2018
New Revision: 258949
URL: https://gcc.gnu.org/viewcvs?rev=258949=gcc=rev
Log:
[ARM][PR target/84826] Fix ICE in extract_insn, at recog.c:2304 on
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
--- Comment #13 from sudi at gcc dot gnu.org ---
Author: sudi
Date: Thu Mar 29 09:19:45 2018
New Revision: 258948
URL: https://gcc.gnu.org/viewcvs?rev=258948=gcc=rev
Log:
[ARM][PR target/84826] Fix ICE in extract_insn, at recog.c:2304 on
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
sudi at gcc dot gnu.org changed:
What|Removed |Added
Known to fail||6.4.1, 7.3.1
--- Comment #12
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
--- Comment #11 from sudi at gcc dot gnu.org ---
Author: sudi
Date: Fri Mar 23 13:57:28 2018
New Revision: 258805
URL: https://gcc.gnu.org/viewcvs?rev=258805=gcc=rev
Log:
[ARM] Fix pr84826.c failure for thumb1
*** gcc/testsuite/ChangeLog ***
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
--- Comment #10 from sudi at gcc dot gnu.org ---
Author: sudi
Date: Thu Mar 22 17:24:41 2018
New Revision: 258777
URL: https://gcc.gnu.org/viewcvs?rev=258777=gcc=rev
Log:
[ARM][PR target/84826] Fix ICE in extract_insn, at recog.c:2304 on
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
--- Comment #9 from sudi at gcc dot gnu.org ---
Proposed patch
https://gcc.gnu.org/ml/gcc-patches/2018-03/msg01120.html
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
--- Comment #8 from sudi at gcc dot gnu.org ---
(In reply to Wilco from comment #5)
> It seems a latent bug in arm_r3_live_at_start_p which now triggers much more
> often due to stack clash protection:
>
> if (IS_NESTED (arm_current_func_type
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
--- Comment #7 from Eric Botcazou ---
> Oops I think I missed the artificial defs. Then the liveness makes sense.
> Out of curiosity why are all the argument register defined? This function
> for instance does not need 4 argument register.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
--- Comment #6 from sudi at gcc dot gnu.org ---
(In reply to Eric Botcazou from comment #4)
> > So I looked into this. Turns out the actual issue is that USE_RETURN_INSN
> > (FALSE) changes its value and becomes false after pass ce3.
> >
> >
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
Wilco changed:
What|Removed |Added
CC||wilco at gcc dot gnu.org
--- Comment #5 from
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
--- Comment #4 from Eric Botcazou ---
> So I looked into this. Turns out the actual issue is that USE_RETURN_INSN
> (FALSE) changes its value and becomes false after pass ce3.
>
> According to what I can see, arm_r3_live_at_start_p() starts to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
sudi at gcc dot gnu.org changed:
What|Removed |Added
CC||ebotcazou at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
sudi at gcc dot gnu.org changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
--- Comment #2 from ktkachov at gcc dot gnu.org ---
Hmmm, the epilogue expansion in arm.md hits the HAVE_return path and does:
emit_jump_insn (gen_return ());
with a comment saying:
/* HAVE_return is testing for USE_RETURN_INSN (FALSE).
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
ktkachov at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
CC|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
Richard Biener changed:
What|Removed |Added
Version|unknown |8.0.1
Target Milestone|8.0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84826
Martin Liška changed:
What|Removed |Added
Target||arm-linux-gnueabi
Last reconfirmed|
18 matches
Mail list logo
