Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-07-03 Thread Richard Sandiford
Jeff Law writes: > On 6/28/19 12:46 PM, Richard Sandiford wrote: >> Segher Boessenkool writes: >>> On Fri, Jun 28, 2019 at 08:55:00AM -0600, Martin Sebor wrote: Jeff reminded me in a code review the other day that GCC does have a guideline for defining POD structs with the keyword

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-07-02 Thread Jeff Law
On 6/28/19 12:46 PM, Richard Sandiford wrote: > Segher Boessenkool writes: >> On Fri, Jun 28, 2019 at 08:55:00AM -0600, Martin Sebor wrote: >>> Jeff reminded me in a code review the other day that GCC does >>> have a guideline for defining POD structs with the keyword >>> "struct" and classes with

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-06-28 Thread Martin Sebor
On 6/28/19 2:10 PM, Segher Boessenkool wrote: On Fri, Jun 28, 2019 at 07:46:54PM +0100, Richard Sandiford wrote: Segher Boessenkool writes: On Fri, Jun 28, 2019 at 08:55:00AM -0600, Martin Sebor wrote: Jeff reminded me in a code review the other day that GCC does have a guideline for defining

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-06-28 Thread Segher Boessenkool
On Fri, Jun 28, 2019 at 07:46:54PM +0100, Richard Sandiford wrote: > Segher Boessenkool writes: > > On Fri, Jun 28, 2019 at 08:55:00AM -0600, Martin Sebor wrote: > >> Jeff reminded me in a code review the other day that GCC does > >> have a guideline for defining POD structs with the keyword > >>

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-06-28 Thread Richard Sandiford
Segher Boessenkool writes: > On Fri, Jun 28, 2019 at 08:55:00AM -0600, Martin Sebor wrote: >> Jeff reminded me in a code review the other day that GCC does >> have a guideline for defining POD structs with the keyword >> "struct" and classes with ctors/dtors using "class": >> >> https://gcc.gnu

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-06-28 Thread Martin Sebor
On 6/28/19 10:59 AM, Jeff Law wrote: On 6/28/19 8:55 AM, Martin Sebor wrote: On 5/28/19 10:31 AM, Martin Sebor wrote: On 5/28/19 4:24 AM, Martin Liška wrote: On 5/28/19 11:31 AM, David CARLIER wrote: Hi, Here a tiny patch to fix few build warnings. Kind regards. Hi. Well, I see a lot of

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-06-28 Thread Jeff Law
On 6/28/19 8:55 AM, Martin Sebor wrote: > On 5/28/19 10:31 AM, Martin Sebor wrote: >> On 5/28/19 4:24 AM, Martin Liška wrote: >>> On 5/28/19 11:31 AM, David CARLIER wrote: Hi, Here a tiny patch to fix few build warnings. Kind regards. >>> >>> Hi. >>> >>> Well, I see a

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-06-28 Thread Segher Boessenkool
On Fri, Jun 28, 2019 at 08:55:00AM -0600, Martin Sebor wrote: > Jeff reminded me in a code review the other day that GCC does > have a guideline for defining POD structs with the keyword > "struct" and classes with ctors/dtors using "class": > > https://gcc.gnu.org/codingconventions.html#Struct_

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-06-28 Thread Martin Sebor
On 5/28/19 10:31 AM, Martin Sebor wrote: On 5/28/19 4:24 AM, Martin Liška wrote: On 5/28/19 11:31 AM, David CARLIER wrote: Hi, Here a tiny patch to fix few build warnings. Kind regards. Hi. Well, I see a lot of these struct/class discrepancies when building GCC with LLVM. Question is wh

Re: Fwd: [PATCH] Fix few build warnings with LLVM toolchain

2019-05-29 Thread Segher Boessenkool
On Tue, May 28, 2019 at 10:28:07AM +, David CARLIER wrote: > -- Forwarded message - > From: David CARLIER > Date: Tue, 28 May 2019 at 10:16 > Subject: Re: [PATCH] Fix few build warnings with LLVM toolchain > To: Segher Boessenkool > > > All right,

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-05-29 Thread Jakub Jelinek
On Wed, May 29, 2019 at 12:53:44PM +0100, David CARLIER wrote: > Here a little progress but maybe it s better doing this in small > "batches" rather than fixing everything in one shot ? IMHO if we want to do anything about this, we should just in system.h add #ifdef __clang__ #pragma clang diagnos

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-05-29 Thread David CARLIER
Here a little progress but maybe it s better doing this in small "batches" rather than fixing everything in one shot ? Kind regards. Fixing few build warnings with clang/clang++ of this type: ../.././gcc/coretypes.h:76:1: warning: class 'rtx_def' was previously declared as a struct; this is valid,

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-05-28 Thread Eric Gallager
On 5/28/19, Martin Sebor wrote: > On 5/28/19 4:24 AM, Martin Liška wrote: >> On 5/28/19 11:31 AM, David CARLIER wrote: >>> Hi, >>> >>> Here a tiny patch to fix few build warnings. >>> >>> Kind regards. >>> >> >> Hi. >> >> Well, I see a lot of these struct/class discrepancies when building GCC >> w

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-05-28 Thread Martin Sebor
On 5/28/19 4:24 AM, Martin Liška wrote: On 5/28/19 11:31 AM, David CARLIER wrote: Hi, Here a tiny patch to fix few build warnings. Kind regards. Hi. Well, I see a lot of these struct/class discrepancies when building GCC with LLVM. Question is whether it worth changing? I think it's nic

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-05-28 Thread David CARLIER
Yes I sort of agree it is pretty harmless even though I cannot tell for Microsoft toolchain issue here. On Tue, 28 May 2019 at 10:27, Jakub Jelinek wrote: > > On Tue, May 28, 2019 at 12:24:12PM +0200, Martin Liška wrote: > > Well, I see a lot of these struct/class discrepancies when building GCC

Fwd: [PATCH] Fix few build warnings with LLVM toolchain

2019-05-28 Thread David CARLIER
-- Forwarded message - From: David CARLIER Date: Tue, 28 May 2019 at 10:16 Subject: Re: [PATCH] Fix few build warnings with LLVM toolchain To: Segher Boessenkool All right, here an updated version, hope it looks better. Thanks. On Tue, 28 May 2019 at 10:09, Segher Boessenkool

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-05-28 Thread Jakub Jelinek
On Tue, May 28, 2019 at 12:24:12PM +0200, Martin Liška wrote: > Well, I see a lot of these struct/class discrepancies when building GCC with > LLVM. > Question is whether it worth changing? No, the warning is just wrong. C++ clearly states what the struct and class keyword mean and it is just fi

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-05-28 Thread Martin Liška
On 5/28/19 11:31 AM, David CARLIER wrote: > Hi, > > Here a tiny patch to fix few build warnings. > > Kind regards. > Hi. Well, I see a lot of these struct/class discrepancies when building GCC with LLVM. Question is whether it worth changing? Martin

Re: [PATCH] Fix few build warnings with LLVM toolchain

2019-05-28 Thread Segher Boessenkool
On Tue, May 28, 2019 at 09:31:18AM +, David CARLIER wrote: > Here a tiny patch to fix few build warnings. Please mention what the warning _is_, and why it is correct / why it is a good idea to make these changes. Segher

[PATCH] Fix few build warnings with LLVM toolchain

2019-05-28 Thread David CARLIER
Hi, Here a tiny patch to fix few build warnings. Kind regards. Index: gcc/ChangeLog === --- gcc/ChangeLog (revision 271684) +++ gcc/ChangeLog (working copy) @@ -1,3 +1,7 @@ +2019-05-28 David Carlier + +* coretypes.h: Fix bu