On Fri, 30 Aug 2024, Jakub Jelinek wrote:

> On Fri, Aug 16, 2024 at 04:58:58PM +0000, Joseph Myers wrote:
> > On Thu, 15 Aug 2024, Jakub Jelinek wrote:
> > 
> > > +   else
> > > +     {
> > > +       if (res > INTTYPE_MAXIMUM (off_t))
> > > +         cpp_error_with_line (pfile, CPP_DL_ERROR, loc, 0,
> > > +                              "too large 'gnu::offset' argument");
> > 
> > Having a testcase for this diagnostic would be a good idea.  Also one for 
> > a negative argument for gnu::offset (the errors for negative arguments are 
> > already tested for limit, but I think testing that for gnu::offset is a 
> > good idea as well).
> 
> Here is an updated patch which does that:

This patch is OK.

-- 
Joseph S. Myers
josmy...@redhat.com

Reply via email to