This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug target/40934] [4.5 Regression] ICE in get_hard_regnum, at reg-stack.c:741
- From: "ubizjak at gmail dot com" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: 2 Aug 2009 09:59:53 -0000
- Subject: [Bug target/40934] [4.5 Regression] ICE in get_hard_regnum, at reg-stack.c:741
- References: <bug-40934-17247@http.gcc.gnu.org/bugzilla/>
- Reply-to: gcc-bugzilla at gcc dot gnu dot org
------- Comment #13 from ubizjak at gmail dot com 2009-08-02 09:59 -------
(In reply to comment #11)
> I'm not sure if there is another patch that can work. :-)
>
> Uros, do you think it's fine? I don't think that non-hot FP jumps are that
> common. Definitely in the minimized testcase it's not a great prediction...
Yes, I think this is OK. optimize_insn_for_size_p () is not stable through RTL
passes, so we either don't split or generate invalid insn in certain cases
(like the one above).
In the above test, we expand via ARITH path, and switch to SAHF somewhere
during optimization passes. Since the comparison in the expanded insn is not
valid for SAHF, it is not split.
And since splitters depend on stable predicates to split expanded sequences, we
have no other choice.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40934