Hi,
patch is OK with change below provided that you add a ChangeLog entry
(it is usual to write those into the emails) and that it passes
bootstrap&regtesting (it is also usual to indicate this).

Do you have rights to the svn repository and copyright assignment done?

>  
> +DEFPARAM (PARAM_IPA_MAX_SWITCH_PREDICATE_BOUNDS,
> +       "ipa-max-switch-predicate-bounds",
> +       "Maximal number of boundary endpoints of case ranges of switch "
> +       "statement.  For switch exceeding this limit, do not construct cost-"
> +       "evaluating predicate for default case during IPA function summary"
> +       "generation.",

I think those texts are intended to not be too long. I think the first
sentence extended by " used during IPA functoin summary generation" is
good enough.

Thank you,
Honza

Reply via email to