Re: [PATCH] diagnostics: fix crash with -fdiagnostics-format=json-file

2023-03-16 Thread David Malcolm via Gcc-patches
On Tue, 2023-01-10 at 16:10 +0100, Martin Liška wrote: > On 1/6/23 14:21, David Malcolm wrote: > > On Fri, 2023-01-06 at 12:33 +0100, Martin Liška wrote: > > > Patch can bootstrap on x86_64-linux-gnu and survives regression > > > tests. > > > > Thanks for the patch. > > > > I noticed that you mar

Re: [PATCH] diagnostics: fix crash with -fdiagnostics-format=json-file

2023-03-16 Thread Martin Liška
PING^5 On 2/27/23 10:49, Martin Liška wrote: > PING^4 > > On 2/17/23 15:52, Martin Liška wrote: >> PING^3 >> >> On 2/1/23 14:13, Martin Liška wrote: >>> PING^2 >>> >>> On 1/24/23 14:34, Martin Liška wrote: PING^1 On 1/10/23 16:10, Martin Liška wrote: > On 1/6/23 14:21, David Ma

Re: [PATCH] diagnostics: fix crash with -fdiagnostics-format=json-file

2023-02-27 Thread Martin Liška
PING^4 On 2/17/23 15:52, Martin Liška wrote: PING^3 On 2/1/23 14:13, Martin Liška wrote: PING^2 On 1/24/23 14:34, Martin Liška wrote: PING^1 On 1/10/23 16:10, Martin Liška wrote: On 1/6/23 14:21, David Malcolm wrote: On Fri, 2023-01-06 at 12:33 +0100, Martin Liška wrote: Patch can bootst

Re: [PATCH] diagnostics: fix crash with -fdiagnostics-format=json-file

2023-02-17 Thread Martin Liška
PING^3 On 2/1/23 14:13, Martin Liška wrote: > PING^2 > > On 1/24/23 14:34, Martin Liška wrote: >> PING^1 >> >> On 1/10/23 16:10, Martin Liška wrote: >>> On 1/6/23 14:21, David Malcolm wrote: On Fri, 2023-01-06 at 12:33 +0100, Martin Liška wrote: > Patch can bootstrap on x86_64-linux-gnu

Re: [PATCH] diagnostics: fix crash with -fdiagnostics-format=json-file

2023-02-01 Thread Martin Liška
PING^2 On 1/24/23 14:34, Martin Liška wrote: PING^1 On 1/10/23 16:10, Martin Liška wrote: On 1/6/23 14:21, David Malcolm wrote: On Fri, 2023-01-06 at 12:33 +0100, Martin Liška wrote: Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Thanks for the patch. I noticed tha

Re: [PATCH] diagnostics: fix crash with -fdiagnostics-format=json-file

2023-01-24 Thread Martin Liška
PING^1 On 1/10/23 16:10, Martin Liška wrote: > On 1/6/23 14:21, David Malcolm wrote: >> On Fri, 2023-01-06 at 12:33 +0100, Martin Liška wrote: >>> Patch can bootstrap on x86_64-linux-gnu and survives regression >>> tests. >> >> Thanks for the patch. >> >> I noticed that you marked PR 108307 as a d

Re: [PATCH] diagnostics: fix crash with -fdiagnostics-format=json-file

2023-01-10 Thread Martin Liška
On 1/6/23 14:21, David Malcolm wrote: > On Fri, 2023-01-06 at 12:33 +0100, Martin Liška wrote: >> Patch can bootstrap on x86_64-linux-gnu and survives regression >> tests. > > Thanks for the patch. > > I noticed that you marked PR 108307 as a dup of this, which covers > -fdiagnostics-format=sarif

Re: [PATCH] diagnostics: fix crash with -fdiagnostics-format=json-file

2023-01-06 Thread David Malcolm via Gcc-patches
On Fri, 2023-01-06 at 12:33 +0100, Martin Liška wrote: > Patch can bootstrap on x86_64-linux-gnu and survives regression > tests. Thanks for the patch. I noticed that you marked PR 108307 as a dup of this, which covers -fdiagnostics-format=sarif-file (and a .S file as input). The patch doesn't a

[PATCH] diagnostics: fix crash with -fdiagnostics-format=json-file

2023-01-06 Thread Martin Liška
Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be installed? Thanks, Martin PR middle-end/106133 gcc/ChangeLog: * diagnostic.cc (diagnostic_output_format_init): If -fdiagnostics-format=json-file and -E is used, then base_file_name