This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][RFC] Minuscule patch to remove apparently accidental break from switch
- From: Simon Baldwin <simonb at google dot com>
- To: Andrew Pinski <pinskia at gmail dot com>
- Cc: Arnaud Charlet <charlet at adacore dot com>, David Daney <ddaney at avtrex dot com>, gcc-patches at gcc dot gnu dot org
- Date: Wed, 01 Aug 2007 16:40:24 -0700
- Subject: Re: [PATCH][RFC] Minuscule patch to remove apparently accidental break from switch
- References: <20070605180327.686E541A711@localhost> <de8d50360706051105s20fdeef4vc7eff5d91c0ab113@mail.gmail.com> <4665BC50.9000000@google.com> <4665BFFD.70805@avtrex.com> <20070606080013.GA12354@adacore.com>
Arnaud Charlet wrote:
Will those cover it, or should I plough through Fortran, ADA, and so on
as well?
Just remove the --enable-languages and test all default languages.
To answer the question about Ada, to enable all languages, including Ada,
you can do: --enable-languages=all,ada
You do not have to, but you are encouraged to, so thanks in advance if
you do.
I've finally got around to rebuilding and retesting for all languages,
including Ada, objective C, and Fortran, with this patch. There is no
difference in test results between the two.
Okay for trunk, then?
BTW, as a refresher, since it's been a while, here's the totality of the
patch:
$ svn diff simplify-rtx.c
Index: simplify-rtx.c
===================================================================
--- simplify-rtx.c (revision 127106)
+++ simplify-rtx.c (working copy)
@@ -2293,8 +2293,6 @@
&& (reversed = reversed_comparison (op0, mode)))
return reversed;
- break;
-
tem = simplify_associative_operation (code, mode, op0, op1);
if (tem)
return tem;