public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug testsuite/53222] New: dejagnu trims leading whitespace
@ 2012-05-03 22:39 manu at gcc dot gnu.org
  2013-04-19  7:29 ` [Bug testsuite/53222] " skannan at redhat dot com
  2013-04-19  7:51 ` manu at gcc dot gnu.org
  0 siblings, 2 replies; 3+ messages in thread
From: manu at gcc dot gnu.org @ 2012-05-03 22:39 UTC (permalink / raw)
  To: gcc-bugs

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53222

             Bug #: 53222
           Summary: dejagnu trims leading whitespace
    Classification: Unclassified
           Product: gcc
           Version: unknown
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: testsuite
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: manu@gcc.gnu.org


This patch should still allow us to prune all caret diagnostics, since they
always start with whitespace:

Index: libstdc++-v3/testsuite/lib/prune.exp
===================================================================
--- libstdc++-v3/testsuite/lib/prune.exp        (revision 187039)
+++ libstdc++-v3/testsuite/lib/prune.exp        (working copy)
@@ -34,11 +34,11 @@ proc libstdc++-dg-prune { system text } 

 #    send_user "Before:$text\n"

     # Ignore caret diagnostics. Unfortunately dejaGNU trims leading
     # spaces, so one cannot rely on them being present.
-    regsub -all "(^|\n)\[^\n\]+\n *\\^\n" $text "\n" text
+    regsub -all "(^ |\n )\[^\n\]+\n *\\^\n" $text "\n" text

     # Cygwin warns about -ffunction-sections
     regsub -all "(^|\n)\[^\n\]*: -ffunction-sections may affect debugging on
some targets\[^\n\]*" $text "" text

     # Remove parts of warnings that refer to location of previous

However, DejaGNU seems to trim leading whitespace:

/home/manuel/test2/src/libstdc++-v3/testsuite/17_intro/using_namespace_std_tr1_neg.cc:66:27:
error: expected namespace-name before ';' token^M
   using namespace std::tr1;  // { dg-error "is not a namespace-name" }^M
                           ^^M

PASS: 17_intro/using_namespace_std_tr1_neg.cc  (test for errors, line 66)
PASS: 17_intro/using_namespace_std_tr1_neg.cc  (test for errors, line 66)
FAIL: 17_intro/using_namespace_std_tr1_neg.cc (test for excess errors)
Excess errors:
using namespace std::tr1;  // { dg-error "is not a namespace-name" }
                        ^

This is unfortunate, because it makes quite hard to match any arbitrary output.


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug testsuite/53222] dejagnu trims leading whitespace
  2012-05-03 22:39 [Bug testsuite/53222] New: dejagnu trims leading whitespace manu at gcc dot gnu.org
@ 2013-04-19  7:29 ` skannan at redhat dot com
  2013-04-19  7:51 ` manu at gcc dot gnu.org
  1 sibling, 0 replies; 3+ messages in thread
From: skannan at redhat dot com @ 2013-04-19  7:29 UTC (permalink / raw)
  To: gcc-bugs


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53222

--- Comment #1 from Shakthi Kannan <skannan at redhat dot com> 2013-04-19 07:29:43 UTC ---
Created attachment 29902
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=29902
namespace error after applying patch

Can you please provide the version of GCC that you used?

I built GCC-4.8.0 (x86_64) after applying the patch and I get the attached
errors.


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [Bug testsuite/53222] dejagnu trims leading whitespace
  2012-05-03 22:39 [Bug testsuite/53222] New: dejagnu trims leading whitespace manu at gcc dot gnu.org
  2013-04-19  7:29 ` [Bug testsuite/53222] " skannan at redhat dot com
@ 2013-04-19  7:51 ` manu at gcc dot gnu.org
  1 sibling, 0 replies; 3+ messages in thread
From: manu at gcc dot gnu.org @ 2013-04-19  7:51 UTC (permalink / raw)
  To: gcc-bugs


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53222

Manuel López-Ibáñez <manu at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |trivial

--- Comment #2 from Manuel López-Ibáñez <manu at gcc dot gnu.org> 2013-04-19 07:51:14 UTC ---
Did you run with -fdiagnostics-show-caret enabled? 

At the end, it was decided to disable it for the testsuite, so this report is
not very important anymore. It would be nice if DejaGNU didn't alter the
output, but right now, it doesn't affect us.
>From gcc-bugs-return-420585-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Fri Apr 19 08:03:38 2013
Return-Path: <gcc-bugs-return-420585-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Delivered-To: listarch-gcc-bugs@gcc.gnu.org
Received: (qmail 10978 invoked by alias); 19 Apr 2013 08:03:38 -0000
Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-bugs.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-help@gcc.gnu.org>
Sender: gcc-bugs-owner@gcc.gnu.org
Delivered-To: mailing list gcc-bugs@gcc.gnu.org
Received: (qmail 10945 invoked by uid 48); 19 Apr 2013 08:03:34 -0000
From: "jakub at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug c++/57000] [4.8/4.9 Regression] ICE with -Ofast and -frounding-math
Date: Fri, 19 Apr 2013 08:03:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: c++
X-Bugzilla-Keywords:
X-Bugzilla-Severity: normal
X-Bugzilla-Who: jakub at gcc dot gnu.org
X-Bugzilla-Status: NEW
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 4.8.1
X-Bugzilla-Changed-Fields:
Message-ID: <bug-57000-4-MDA25hQXVB@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-57000-4@http.gcc.gnu.org/bugzilla/>
References: <bug-57000-4@http.gcc.gnu.org/bugzilla/>
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
Content-Type: text/plain; charset="UTF-8"
MIME-Version: 1.0
X-SW-Source: 2013-04/txt/msg01730.txt.bz2
Content-length: 372


http://gcc.gnu.org/bugzilla/show_bug.cgi?idW000

--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> 2013-04-19 08:03:34 UTC ---
Shorter testcase:
/* PR tree-optimization/57000 */
/* { dg-do compile } */
/* { dg-options "-Ofast -frounding-math" } */

double baz (double foo, double bar)
{
  return foo * foo * foo * foo * bar * bar * bar * bar;
}


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-04-19  7:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-05-03 22:39 [Bug testsuite/53222] New: dejagnu trims leading whitespace manu at gcc dot gnu.org
2013-04-19  7:29 ` [Bug testsuite/53222] " skannan at redhat dot com
2013-04-19  7:51 ` manu at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).