diff mbox series

libgccjit: Fix a failing test

Message ID CAAOQCfRj26CgWLBq=9M4AsC5WhhXOW4s3ynbjoSt2n8mv6LbRA@mail.gmail.com
State Accepted
Headers
Series libgccjit: Fix a failing test |

Checks

Context Check Description
snail/gcc-patch-check success Github commit url

Commit Message

Guillaume Gomez Dec. 14, 2022, 1:39 p.m. UTC 
  Hi,

This fixes bug 107999.

Thanks in advance for the review.

Comments

Antoni Boucher Dec. 14, 2022, 3:12 p.m. UTC | #1 
Thanks!

In your patch, you're missing this line at the end of the commit
message:

   Signed-off-by: Guillaume Gomez <guillaume1.gomez@gmail.com>

On Wed, 2022-12-14 at 14:39 +0100, Guillaume Gomez via Jit wrote:
> Hi,
> 
> This fixes bug 107999.
> 
> Thanks in advance for the review.
Guillaume Gomez Dec. 15, 2022, 7:34 a.m. UTC | #2 
Forgot it indeed, thanks for notifying me!

I modified the commit message to add it and added it into this email.

Le mer. 14 déc. 2022 à 16:12, Antoni Boucher <bouanto@zoho.com> a écrit :

> Thanks!
>
> In your patch, you're missing this line at the end of the commit
> message:
>
>    Signed-off-by: Guillaume Gomez <guillaume1.gomez@gmail.com>
>
> On Wed, 2022-12-14 at 14:39 +0100, Guillaume Gomez via Jit wrote:
> > Hi,
> >
> > This fixes bug 107999.
> >
> > Thanks in advance for the review.
>
>
Guillaume Gomez Dec. 24, 2022, 8:01 p.m. UTC | #3 
Ping David

Le jeu. 15 déc. 2022 à 11:34, Guillaume Gomez <guillaume1.gomez@gmail.com>
a écrit :

> Forgot it indeed, thanks for notifying me!
>
> I modified the commit message to add it and added it into this email.
>
> Le mer. 14 déc. 2022 à 16:12, Antoni Boucher <bouanto@zoho.com> a écrit :
>
>> Thanks!
>>
>> In your patch, you're missing this line at the end of the commit
>> message:
>>
>>    Signed-off-by: Guillaume Gomez <guillaume1.gomez@gmail.com>
>>
>> On Wed, 2022-12-14 at 14:39 +0100, Guillaume Gomez via Jit wrote:
>> > Hi,
>> >
>> > This fixes bug 107999.
>> >
>> > Thanks in advance for the review.
>>
>>
Guillaume Gomez Jan. 5, 2023, 10:37 p.m. UTC | #4 
Ping David.

Le sam. 24 déc. 2022 à 21:01, Guillaume Gomez <guillaume1.gomez@gmail.com>
a écrit :

> Ping David
>
> Le jeu. 15 déc. 2022 à 11:34, Guillaume Gomez <guillaume1.gomez@gmail.com>
> a écrit :
>
>> Forgot it indeed, thanks for notifying me!
>>
>> I modified the commit message to add it and added it into this email.
>>
>> Le mer. 14 déc. 2022 à 16:12, Antoni Boucher <bouanto@zoho.com> a écrit :
>>
>>> Thanks!
>>>
>>> In your patch, you're missing this line at the end of the commit
>>> message:
>>>
>>>    Signed-off-by: Guillaume Gomez <guillaume1.gomez@gmail.com>
>>>
>>> On Wed, 2022-12-14 at 14:39 +0100, Guillaume Gomez via Jit wrote:
>>> > Hi,
>>> >
>>> > This fixes bug 107999.
>>> >
>>> > Thanks in advance for the review.
>>>
>>>
Guillaume Gomez Jan. 16, 2023, 2:08 p.m. UTC | #5 
Ping David.

Le jeu. 5 janv. 2023 à 23:37, Guillaume Gomez <guillaume1.gomez@gmail.com>
a écrit :

> Ping David.
>
> Le sam. 24 déc. 2022 à 21:01, Guillaume Gomez <guillaume1.gomez@gmail.com>
> a écrit :
>
>> Ping David
>>
>> Le jeu. 15 déc. 2022 à 11:34, Guillaume Gomez <guillaume1.gomez@gmail.com>
>> a écrit :
>>
>>> Forgot it indeed, thanks for notifying me!
>>>
>>> I modified the commit message to add it and added it into this email.
>>>
>>> Le mer. 14 déc. 2022 à 16:12, Antoni Boucher <bouanto@zoho.com> a
>>> écrit :
>>>
>>>> Thanks!
>>>>
>>>> In your patch, you're missing this line at the end of the commit
>>>> message:
>>>>
>>>>    Signed-off-by: Guillaume Gomez <guillaume1.gomez@gmail.com>
>>>>
>>>> On Wed, 2022-12-14 at 14:39 +0100, Guillaume Gomez via Jit wrote:
>>>> > Hi,
>>>> >
>>>> > This fixes bug 107999.
>>>> >
>>>> > Thanks in advance for the review.
>>>>
>>>>
Guillaume Gomez March 2, 2023, 7:41 p.m. UTC | #6 
Ping David.

Le lun. 16 janv. 2023 à 15:08, Guillaume Gomez <guillaume1.gomez@gmail.com>
a écrit :

> Ping David.
>
> Le jeu. 5 janv. 2023 à 23:37, Guillaume Gomez <guillaume1.gomez@gmail.com>
> a écrit :
>
>> Ping David.
>>
>> Le sam. 24 déc. 2022 à 21:01, Guillaume Gomez <guillaume1.gomez@gmail.com>
>> a écrit :
>>
>>> Ping David
>>>
>>> Le jeu. 15 déc. 2022 à 11:34, Guillaume Gomez <
>>> guillaume1.gomez@gmail.com> a écrit :
>>>
>>>> Forgot it indeed, thanks for notifying me!
>>>>
>>>> I modified the commit message to add it and added it into this email.
>>>>
>>>> Le mer. 14 déc. 2022 à 16:12, Antoni Boucher <bouanto@zoho.com> a
>>>> écrit :
>>>>
>>>>> Thanks!
>>>>>
>>>>> In your patch, you're missing this line at the end of the commit
>>>>> message:
>>>>>
>>>>>    Signed-off-by: Guillaume Gomez <guillaume1.gomez@gmail.com>
>>>>>
>>>>> On Wed, 2022-12-14 at 14:39 +0100, Guillaume Gomez via Jit wrote:
>>>>> > Hi,
>>>>> >
>>>>> > This fixes bug 107999.
>>>>> >
>>>>> > Thanks in advance for the review.
>>>>>
>>>>>
David Malcolm March 2, 2023, 9:58 p.m. UTC | #7 
On Thu, 2022-12-15 at 08:34 +0100, Guillaume Gomez via Jit wrote:
> Forgot it indeed, thanks for notifying me!
> 
> I modified the commit message to add it and added it into this email.

Sorry about the delay in reviewing this; for some reason I didn't see
the mail.

The patch looks good for trunk, but please add a reference to
  PR jit/107999
to the subject line and ChangeLog message.

Dave

> 
> Le mer. 14 déc. 2022 à 16:12, Antoni Boucher <bouanto@zoho.com> a
> écrit :
> 
> > Thanks!
> > 
> > In your patch, you're missing this line at the end of the commit
> > message:
> > 
> >    Signed-off-by: Guillaume Gomez <guillaume1.gomez@gmail.com>
> > 
> > On Wed, 2022-12-14 at 14:39 +0100, Guillaume Gomez via Jit wrote:
> > > Hi,
> > > 
> > > This fixes bug 107999.
> > > 
> > > Thanks in advance for the review.
> > 
> >
Guillaume Gomez March 2, 2023, 10:19 p.m. UTC | #8 
No problem, thanks for the explanations.

I joined the patch with the fixed commit message.

Le jeu. 2 mars 2023 à 22:58, David Malcolm <dmalcolm@redhat.com> a écrit :

> On Thu, 2022-12-15 at 08:34 +0100, Guillaume Gomez via Jit wrote:
> > Forgot it indeed, thanks for notifying me!
> >
> > I modified the commit message to add it and added it into this email.
>
> Sorry about the delay in reviewing this; for some reason I didn't see
> the mail.
>
> The patch looks good for trunk, but please add a reference to
>   PR jit/107999
> to the subject line and ChangeLog message.
>
> Dave
>
> >
> > Le mer. 14 déc. 2022 à 16:12, Antoni Boucher <bouanto@zoho.com> a
> > écrit :
> >
> > > Thanks!
> > >
> > > In your patch, you're missing this line at the end of the commit
> > > message:
> > >
> > >    Signed-off-by: Guillaume Gomez <guillaume1.gomez@gmail.com>
> > >
> > > On Wed, 2022-12-14 at 14:39 +0100, Guillaume Gomez via Jit wrote:
> > > > Hi,
> > > >
> > > > This fixes bug 107999.
> > > >
> > > > Thanks in advance for the review.
> > >
> > >
>
>
Guillaume Gomez March 2, 2023, 10:29 p.m. UTC | #9 
Just realized I used whitespace and not a tab. Sorry about that. Here's the
fixed version...

Le jeu. 2 mars 2023 à 23:19, Guillaume Gomez <guillaume1.gomez@gmail.com> a
écrit :

> No problem, thanks for the explanations.
>
> I joined the patch with the fixed commit message.
>
> Le jeu. 2 mars 2023 à 22:58, David Malcolm <dmalcolm@redhat.com> a écrit :
>
>> On Thu, 2022-12-15 at 08:34 +0100, Guillaume Gomez via Jit wrote:
>> > Forgot it indeed, thanks for notifying me!
>> >
>> > I modified the commit message to add it and added it into this email.
>>
>> Sorry about the delay in reviewing this; for some reason I didn't see
>> the mail.
>>
>> The patch looks good for trunk, but please add a reference to
>>   PR jit/107999
>> to the subject line and ChangeLog message.
>>
>> Dave
>>
>> >
>> > Le mer. 14 déc. 2022 à 16:12, Antoni Boucher <bouanto@zoho.com> a
>> > écrit :
>> >
>> > > Thanks!
>> > >
>> > > In your patch, you're missing this line at the end of the commit
>> > > message:
>> > >
>> > >    Signed-off-by: Guillaume Gomez <guillaume1.gomez@gmail.com>
>> > >
>> > > On Wed, 2022-12-14 at 14:39 +0100, Guillaume Gomez via Jit wrote:
>> > > > Hi,
>> > > >
>> > > > This fixes bug 107999.
>> > > >
>> > > > Thanks in advance for the review.
>> > >
>> > >
>>
>>
David Malcolm March 2, 2023, 10:33 p.m. UTC | #10 
On Thu, 2023-03-02 at 23:29 +0100, Guillaume Gomez wrote:
> Just realized I used whitespace and not a tab. Sorry about that.
> Here's the
> fixed version...

Looks great.  Do you have push rights, or do you want me to push this?

Thanks
Dave

> 
> Le jeu. 2 mars 2023 à 23:19, Guillaume Gomez
> <guillaume1.gomez@gmail.com> a
> écrit :
> 
> > No problem, thanks for the explanations.
> > 
> > I joined the patch with the fixed commit message.
> > 
> > Le jeu. 2 mars 2023 à 22:58, David Malcolm <dmalcolm@redhat.com> a
> > écrit :
> > 
> > > On Thu, 2022-12-15 at 08:34 +0100, Guillaume Gomez via Jit wrote:
> > > > Forgot it indeed, thanks for notifying me!
> > > > 
> > > > I modified the commit message to add it and added it into this
> > > > email.
> > > 
> > > Sorry about the delay in reviewing this; for some reason I didn't
> > > see
> > > the mail.
> > > 
> > > The patch looks good for trunk, but please add a reference to
> > >   PR jit/107999
> > > to the subject line and ChangeLog message.
> > > 
> > > Dave
> > > 
> > > > 
> > > > Le mer. 14 déc. 2022 à 16:12, Antoni Boucher <bouanto@zoho.com>
> > > > a
> > > > écrit :
> > > > 
> > > > > Thanks!
> > > > > 
> > > > > In your patch, you're missing this line at the end of the
> > > > > commit
> > > > > message:
> > > > > 
> > > > >    Signed-off-by: Guillaume Gomez
> > > > > <guillaume1.gomez@gmail.com>
> > > > > 
> > > > > On Wed, 2022-12-14 at 14:39 +0100, Guillaume Gomez via Jit
> > > > > wrote:
> > > > > > Hi,
> > > > > > 
> > > > > > This fixes bug 107999.
> > > > > > 
> > > > > > Thanks in advance for the review.
> > > > > 
> > > > > 
> > > 
> > >
Guillaume Gomez March 2, 2023, 10:35 p.m. UTC | #11 
I don't have push rights so if you could push it, it'd be super appreciated!

Le jeu. 2 mars 2023 à 23:33, David Malcolm <dmalcolm@redhat.com> a écrit :

> On Thu, 2023-03-02 at 23:29 +0100, Guillaume Gomez wrote:
> > Just realized I used whitespace and not a tab. Sorry about that.
> > Here's the
> > fixed version...
>
> Looks great.  Do you have push rights, or do you want me to push this?
>
> Thanks
> Dave
>
> >
> > Le jeu. 2 mars 2023 à 23:19, Guillaume Gomez
> > <guillaume1.gomez@gmail.com> a
> > écrit :
> >
> > > No problem, thanks for the explanations.
> > >
> > > I joined the patch with the fixed commit message.
> > >
> > > Le jeu. 2 mars 2023 à 22:58, David Malcolm <dmalcolm@redhat.com> a
> > > écrit :
> > >
> > > > On Thu, 2022-12-15 at 08:34 +0100, Guillaume Gomez via Jit wrote:
> > > > > Forgot it indeed, thanks for notifying me!
> > > > >
> > > > > I modified the commit message to add it and added it into this
> > > > > email.
> > > >
> > > > Sorry about the delay in reviewing this; for some reason I didn't
> > > > see
> > > > the mail.
> > > >
> > > > The patch looks good for trunk, but please add a reference to
> > > >   PR jit/107999
> > > > to the subject line and ChangeLog message.
> > > >
> > > > Dave
> > > >
> > > > >
> > > > > Le mer. 14 déc. 2022 à 16:12, Antoni Boucher <bouanto@zoho.com>
> > > > > a
> > > > > écrit :
> > > > >
> > > > > > Thanks!
> > > > > >
> > > > > > In your patch, you're missing this line at the end of the
> > > > > > commit
> > > > > > message:
> > > > > >
> > > > > >    Signed-off-by: Guillaume Gomez
> > > > > > <guillaume1.gomez@gmail.com>
> > > > > >
> > > > > > On Wed, 2022-12-14 at 14:39 +0100, Guillaume Gomez via Jit
> > > > > > wrote:
> > > > > > > Hi,
> > > > > > >
> > > > > > > This fixes bug 107999.
> > > > > > >
> > > > > > > Thanks in advance for the review.
> > > > > >
> > > > > >
> > > >
> > > >
>
>
David Malcolm March 2, 2023, 10:54 p.m. UTC | #12 
On Thu, 2023-03-02 at 23:35 +0100, Guillaume Gomez wrote:
> I don't have push rights so if you could push it, it'd be super
> appreciated!

Done, as r13-6425-g6b432c0f777ab9; I took the liberty of slightly
tweaking the subject line to add a "jit, testsuite: " prefix.

Thanks again for the patch
Dave
Guillaume Gomez March 3, 2023, 9:31 a.m. UTC | #13 
Thanks!

Le jeu. 2 mars 2023 à 23:54, David Malcolm <dmalcolm@redhat.com> a écrit :

> On Thu, 2023-03-02 at 23:35 +0100, Guillaume Gomez wrote:
> > I don't have push rights so if you could push it, it'd be super
> > appreciated!
>
> Done, as r13-6425-g6b432c0f777ab9; I took the liberty of slightly
> tweaking the subject line to add a "jit, testsuite: " prefix.
>
> Thanks again for the patch
> Dave
>
>
diff mbox series

Patch

From e6db0cb107e54789095f4585dd279a2c984d2ca1 Mon Sep 17 00:00:00 2001
From: Guillaume Gomez <guillaume1.gomez@gmail.com>
Date: Wed, 14 Dec 2022 14:28:22 +0100
Subject: [PATCH] Fix a failing test by updating its error string.

gcc/testsuite/ChangeLog:

	* jit.dg/test-error-array-bounds.c: Update test.
---
 gcc/testsuite/jit.dg/test-error-array-bounds.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/testsuite/jit.dg/test-error-array-bounds.c b/gcc/testsuite/jit.dg/test-error-array-bounds.c
index b6c0ee526d4..a0dead13cb7 100644
--- a/gcc/testsuite/jit.dg/test-error-array-bounds.c
+++ b/gcc/testsuite/jit.dg/test-error-array-bounds.c
@@ -70,5 +70,5 @@  verify_code (gcc_jit_context *ctxt, gcc_jit_result *result)
   /* ...and that the message was captured by the API.  */
   CHECK_STRING_VALUE (gcc_jit_context_get_first_error (ctxt),
 		      "array subscript 10 is above array bounds of"
-		      " 'char[10]' [-Warray-bounds]");
+		      " 'char[10]' [-Warray-bounds=]");
 }
-- 
2.34.1