New options to disable/enable any pass for any functions (issue4550056)
Joseph S. Myers
joseph@codesourcery.com
Wed May 18 20:21:00 GMT 2011
On Wed, 18 May 2011, David Li wrote:
> + error ("Unrecognized option %s", is_enable ? "-fenable" : "-fdisable");
> + error ("Unknown pass %s specified in %s",
> + phase_name,
> + is_enable ? "-fenable" : "-fdisable");
Follow GNU Coding Standards for diagnostics (start with lowercase letter).
> + inform (UNKNOWN_LOCATION, "%s pass %s for functions in the range of [%u, %u]\n",
> + is_enable? "Enable":"Disable", phase_name, new_range->start, new_range->last);
Use separate calls to inform for the enable and disable cases, so that
full sentences can be extracted for translation.
> + error ("Invalid range %s in option %s",
> + one_range,
> + is_enable ? "-fenable" : "-fdisable");
GNU Coding Standards.
> + error ("Invalid range %s in option %s",
Likewise.
> + inform (UNKNOWN_LOCATION, "%s pass %s for functions in the range of [%u, %u]\n",
> + is_enable? "Enable":"Disable", phase_name, new_range->start, new_range->last);
Again needs GCS and i18n fixes.
--
Joseph S. Myers
joseph@codesourcery.com
More information about the Gcc-patches
mailing list