>    /* 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

Reply via email to