Re: [PATCH] libsanitizer: remove -pedantic option

2021-11-05 Thread Martin Liška
On 11/5/21 16:29, Jakub Jelinek wrote: On Fri, Nov 05, 2021 at 04:25:53PM +0100, Martin Liška wrote: On 11/5/21 16:22, H.J. Lu wrote: Should we add __extension__ here? I tried doing that but it didn't help me with the warning. Maybe I did something wrong? Works for me just fine say on: void

Re: [PATCH] libsanitizer: remove -pedantic option

2021-11-05 Thread H.J. Lu via Gcc-patches
On Fri, Nov 5, 2021 at 8:25 AM Martin Liška wrote: > > On 11/5/21 16:22, H.J. Lu wrote: > > Should we add __extension__ here? > > I tried doing that but it didn't help me with the warning. > Maybe I did something wrong? [hjl@gnu-cfl-2 tmp]$ cat y.cc #include #define uptr uintptr_t # define GE

Re: [PATCH] libsanitizer: remove -pedantic option

2021-11-05 Thread Jakub Jelinek via Gcc-patches
On Fri, Nov 05, 2021 at 04:25:53PM +0100, Martin Liška wrote: > On 11/5/21 16:22, H.J. Lu wrote: > > Should we add __extension__ here? > > I tried doing that but it didn't help me with the warning. > Maybe I did something wrong? Works for me just fine say on: void foo () { int a = ({ int d = 1;

Re: [PATCH] libsanitizer: remove -pedantic option

2021-11-05 Thread Martin Liška
On 11/5/21 16:22, H.J. Lu wrote: Should we add __extension__ here? I tried doing that but it didn't help me with the warning. Maybe I did something wrong? Cheers, Martin

Re: [PATCH] libsanitizer: remove -pedantic option

2021-11-05 Thread H.J. Lu via Gcc-patches
On Fri, Nov 5, 2021 at 8:00 AM Martin Liška wrote: > > The code uses intentionally braced-groups within expressions: > > ({\ Should we add __extension__ here? >uptr pc;\ >asm("lea 0(%%rip), %0" : "=r"(pc)); \ >

[PATCH] libsanitizer: remove -pedantic option

2021-11-05 Thread Martin Liška
The code uses intentionally braced-groups within expressions: ({\ uptr pc;\ asm("lea 0(%%rip), %0" : "=r"(pc)); \ pc; \ }) And we emit gazillion of warnings now: /home/marxi