Re: Ping---[V3][PATCH 2/2] Add a new warning option -Wstrict-flex-arrays.

FYI. Committed this last patch as: https://jira.oci.oraclecorp.com/browse/OLDIS-21095 I will come up with the update to gcc-13/changes.html for -fstrict-flex-arrays very soon. Thanks. Qing > On Dec 16, 2022, at 9:49 AM, Qing Zhao via Gcc-patches > wrote: > > > >> On Dec 16, 2022, at 4:17

Re: Ping---[V3][PATCH 2/2] Add a new warning option -Wstrict-flex-arrays.

> On Dec 16, 2022, at 4:17 AM, Richard Biener wrote: > > On Thu, 15 Dec 2022, Qing Zhao wrote: > >> >> >>> On Dec 15, 2022, at 2:47 AM, Richard Biener wrote: >>> >>> On Wed, 14 Dec 2022, Qing Zhao wrote: >>> Hi, Richard, I guess that we now agreed on the following:

Re: Ping---[V3][PATCH 2/2] Add a new warning option -Wstrict-flex-arrays.

On Thu, 15 Dec 2022, Qing Zhao wrote: > > > > On Dec 15, 2022, at 2:47 AM, Richard Biener wrote: > > > > On Wed, 14 Dec 2022, Qing Zhao wrote: > > > >> Hi, Richard, > >> > >> I guess that we now agreed on the following: > >> > >> “ the information that we ran into a trailing array but

Re: Ping---[V3][PATCH 2/2] Add a new warning option -Wstrict-flex-arrays.

> On Dec 15, 2022, at 2:47 AM, Richard Biener wrote: > > On Wed, 14 Dec 2022, Qing Zhao wrote: > >> Hi, Richard, >> >> I guess that we now agreed on the following: >> >> “ the information that we ran into a trailing array but didn't consider >> it a flex array because of

Re: Ping---[V3][PATCH 2/2] Add a new warning option -Wstrict-flex-arrays.

On Wed, 14 Dec 2022, Qing Zhao wrote: > Hi, Richard, > > I guess that we now agreed on the following: > > “ the information that we ran into a trailing array but didn't consider > it a flex array because of -fstrict-flex-arrays is always a useful > information” > > The only thing we didn’t

Re: Ping---[V3][PATCH 2/2] Add a new warning option -Wstrict-flex-arrays.

Hi, Richard, I guess that we now agreed on the following: “ the information that we ran into a trailing array but didn't consider it a flex array because of -fstrict-flex-arrays is always a useful information” The only thing we didn’t decide is: A. Amend such new information to -Warray-bounds

Re: Ping---[V3][PATCH 2/2] Add a new warning option -Wstrict-flex-arrays.

> On Dec 14, 2022, at 9:08 AM, Qing Zhao via Gcc-patches > wrote: > > > >> On Dec 14, 2022, at 4:03 AM, Richard Biener wrote: >> >> On Tue, 13 Dec 2022, Qing Zhao wrote: >> >>> Richard, >>> >>> Do you have any decision on this one? >>> Do we need this warning option For GCC? >> >>

Re: Ping---[V3][PATCH 2/2] Add a new warning option -Wstrict-flex-arrays.

> On Dec 14, 2022, at 4:03 AM, Richard Biener wrote: > > On Tue, 13 Dec 2022, Qing Zhao wrote: > >> Richard, >> >> Do you have any decision on this one? >> Do we need this warning option For GCC? > > Looking at the testcases it seems that the diagnostic amends > -Warray-bounds

Re: Ping---[V3][PATCH 2/2] Add a new warning option -Wstrict-flex-arrays.

On Tue, 13 Dec 2022, Qing Zhao wrote: > Richard, > > Do you have any decision on this one? > Do we need this warning option For GCC? Looking at the testcases it seems that the diagnostic amends -Warray-bounds diagnostics for trailing but not flexible arrays? Wouldn't it be better to

Ping---[V3][PATCH 2/2] Add a new warning option -Wstrict-flex-arrays.

Richard, Do you have any decision on this one? Do we need this warning option For GCC? thanks. Qing > On Dec 6, 2022, at 11:18 AM, Qing Zhao wrote: > > '-Wstrict-flex-arrays' > Warn about inproper usages of flexible array members according to > the LEVEL of the 'strict_flex_array

[V3][PATCH 2/2] Add a new warning option -Wstrict-flex-arrays.

'-Wstrict-flex-arrays' Warn about inproper usages of flexible array members according to the LEVEL of the 'strict_flex_array (LEVEL)' attribute attached to the trailing array field of a structure if it's available, otherwise according to the LEVEL of the option