> /* If we ever encounter an insn without an insn reservation, trip > an assert so we can find and fix this problem. */ > -#if 0 > + if (! insn_has_dfa_reservation_p (insn)) { > + print_rtl(stderr, insn); > + fprintf(stderr, "%d", get_attr_type (insn)); > + } > gcc_assert (insn_has_dfa_reservation_p (insn)); > -#endif > > return more - 1; > }
I was thinking about make the gcc_assert a gcc_checking_assert so, in case we accidentally forget something at any point, it would only gracefully degrade in a release build. As we already have a hard assert for the type the patch (and not many test with enable checking anyway) this is OK IMHO. I suppose you tested with all available -mtune options? Regards Robin