2015-03-12 12:02 GMT+03:00 Jakub Jelinek <ja...@redhat.com>:
> On Thu, Mar 12, 2015 at 11:51:51AM +0300, Ilya Enkovich wrote:
>> On 09 Mar 15:51, Jakub Jelinek wrote:
>> > On Mon, Mar 02, 2015 at 01:25:43PM +0300, Ilya Enkovich wrote:
>> > > > --- a/gcc/toplev.c
>> > > > +++ b/gcc/toplev.c
>> > > > @@ -1376,6 +1376,11 @@ process_options (void)
>> > > >      {
>> > > >        if (targetm.chkp_bound_mode () == VOIDmode)
>> > > >         error ("-fcheck-pointer-bounds is not supported for this 
>> > > > target");
>> > > > +
>> > > > +      if (flag_sanitize & SANITIZE_ADDRESS)
>> > > > +       error ("-fcheck-pointer-bounds is not supported with Address 
>> > > > Sanitizer");
>> > > > +
>> > > > +      flag_check_pointer_bounds = 0;
>> > > >      }
>> >
>> > Doesn't this disable -fcheck-pointer-bounds always?
>> > I'd expect you want to clear flag_check_pointer_bounds only if you issued
>> > one of the two errors...
>> >
>> >     Jakub
>>
>> Whoops!  Here is a less destructive version.
>
> Ok for trunk.  Did the old version pass make check?  If so, perhaps you want 
> to add
> (incrementally) some test that would actually verify that
> -fcheck-pointer-bounds does what it should do (e.g. by scanning tree dumps
> etc.).

Thanks!  I sent previous version before make check.  There are several
chkp tests which would fail.

Ilya

>
>         Jakub

Reply via email to