On Mon, 21 Oct 2013, Ilya Enkovich wrote:

> +  if (flag_check_pointers)
> +    {
> +      if (flag_lto)
> +     sorry ("Pointers checker is not yet fully supported for link-time 
> optimization");

That sounds wrong.  It suggests some bug somewhere in your patch series 
failing to allow for LTO, which should be fixed.  At least give a more 
detailed explanation in a comment of what would need to change to remove 
this call to sorry ().

> +      if (targetm.chkp_bound_mode () == VOIDmode)
> +     error ("-fcheck-pointers is not supported for this target.");

Also note GNU Coding Standards for diagnostics.  They should not start 
with uppercase letters or end with ".".

-- 
Joseph S. Myers
jos...@codesourcery.com

Reply via email to