This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Fix spurious match testsuite regressions from "[PATCH, middle end]: Introduce BUILT_IN_I{CEIL_FLOOR_ROUND_RINT} FP-to-int conversion functions"
- From: Hans-Peter Nilsson <hans-peter dot nilsson at axis dot com>
- To: rguenther at suse dot de
- Cc: hp at axis dot com, gcc-patches at gcc dot gnu dot org, ubizjak at gmail dot com
- Date: Mon, 15 Aug 2011 12:09:40 +0200
- Subject: Re: Fix spurious match testsuite regressions from "[PATCH, middle end]: Introduce BUILT_IN_I{CEIL_FLOOR_ROUND_RINT} FP-to-int conversion functions"
> Date: Mon, 15 Aug 2011 10:43:35 +0200 (CEST)
> From: Richard Guenther <rguenther@suse.de>
> On Mon, 15 Aug 2011, Hans-Peter Nilsson wrote:
> > So, ok as is?
> > If not, would you preapprove tree-dumping decl_uids only at a
> > higher dump verbosity level?
>
> Ok. Yes, definitely, a patch to do decl_uids dumping only at
> TDF_UID is pre-approved (you'd need to adjust some testcases
> I guess).
Bah, should've looked closer; there already was a TDF_NUID gate
there... Looks like this is a problem that is supposed to be
handled by appending -nouid to the dump option. See asm-1.c
which is the only other tree-ssa instance grepping for a number.
So, the following is therefore committed as obvious:
testsuite:
* gcc.dg/tree-ssa/vrp61.c: : Use -fdump-tree-vrp1-nouid instead of
-fdump-tree-vrp1.
Index: gcc.dg/tree-ssa/vrp61.c
===================================================================
--- gcc.dg/tree-ssa/vrp61.c (revision 177757)
+++ gcc.dg/tree-ssa/vrp61.c (working copy)
@@ -1,5 +1,5 @@
/* { dg-do compile } */
-/* { dg-options "-O2 -fdump-tree-vrp1" } */
+/* { dg-options "-O2 -fdump-tree-vrp1-nouid" } */
int f (int x, int y)
{
brgds, H-P