This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH][RFC] Add a subset of -Warray-bounds warnings to C/C++ front ends


On Fri, May 2, 2008 at 12:53 PM, Andrew Pinski <pinskia@gmail.com> wrote:
> On Fri, May 2, 2008 at 9:34 AM, Mark Mitchell <mark@codesourcery.com> wrote:
>  > Simon Baldwin wrote:
>  >
>  >
>  > > I think the wisest course for now is to roll back this change until I have
>  > a better understanding of the issues.
>  > >
>  >
>  >  If you want to roll back, that's fine.  You can do that without any further
>  > approval.
>
>  x86-darwin is also broken with the following warning:
>  /home/regress/tbox/svn-gcc/gcc/config/i386/i386.c:20877: error: array
>
> subscript is above array bounds
>
>  which is a false warning.
>  We have:
>  mode0 = insn_data[icode].operand[0].mode;
>
>  where insn_data is defined as:
>  extern const struct insn_data insn_data[];
>
>  operand is a pointer so that is not an issue.
>

May I suggest to revert it now and use a branch to implement it
properly before merging it with trunk? I can prepare a patch in half
an hour.

Thanks.

H.J.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]