From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 15231 invoked by alias); 25 Apr 2014 03:23:57 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 15216 invoked by uid 89); 25 Apr 2014 03:23:56 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.3 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-vc0-f175.google.com Received: from mail-vc0-f175.google.com (HELO mail-vc0-f175.google.com) (209.85.220.175) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Fri, 25 Apr 2014 03:23:54 +0000 Received: by mail-vc0-f175.google.com with SMTP id lh14so4038132vcb.34 for ; Thu, 24 Apr 2014 20:23:52 -0700 (PDT) MIME-Version: 1.0 X-Received: by 10.52.241.106 with SMTP id wh10mr3698272vdc.16.1398396232545; Thu, 24 Apr 2014 20:23:52 -0700 (PDT) Received: by 10.58.186.176 with HTTP; Thu, 24 Apr 2014 20:23:52 -0700 (PDT) In-Reply-To: <21336.14486.650610.274638@ruffy.mtv.corp.google.com> References: <21336.14486.650610.274638@ruffy.mtv.corp.google.com> Date: Fri, 25 Apr 2014 03:23:00 -0000 Message-ID: Subject: Re: [patch] [gdb/testsuite] XFAIL under Clang tests using label debug info From: David Blaikie To: Doug Evans , Pedro Alves Cc: gdb-patches Content-Type: text/plain; charset=UTF-8 X-IsSubscribed: yes X-SW-Source: 2014-04/txt/msg00523.txt.bz2 On Wed, Apr 23, 2014 at 3:03 PM, Doug Evans wrote: > David Blaikie writes: > > Clang doesn't emit debug info for labels (Clang PR14500). XFAIL a > > bunch of GDB tests that rely on debug info for labels. > > > > For some reason gdb.linespec/ls-expr.exp gathered all tests into a > > dictionary and then ran them. This made it hard to XFAIL just the > > right tests. I refactored this to execute the tests directly, removing > > the dictionary so I could XFAIL the right tests. Is there a reason it > > would've been written that way? Does my patch break it in some way? > > commit c438cb16b63292e415330f289616c4e4ecece63c > > Author: David Blaikie > > Date: Sun Apr 13 11:42:02 2014 -0700 > > > > XFAIL under Clang tests using labels > > > > gdb/testsuite/ > > * gdb.base/label.exp: XFAIL label related tests under Clang. > > * gdb.cp/cplabel.exp: Ditto. > > * gdb.linespec/ls-errs.exp: Refactor tests to execute directly > > and XFAIL under Clang those using labels. > > LGTM > > > diff --git gdb/testsuite/ChangeLog gdb/testsuite/ChangeLog > > index 730c116..b04b940 100644 > > --- gdb/testsuite/ChangeLog > > +++ gdb/testsuite/ChangeLog > > @@ -1,3 +1,9 @@ > > +2014-04-12 David Blaikie > > + > > + * gdb.base/label.exp: XFAIL label related tests under Clang. > > + * gdb.cp/cplabel.exp: Ditto. > > + * gdb.linespec/ls-errs.exp: Refactor tests to execute directly and XFAIL under Clang those using labels. > > + > > Nit: space vs tabs. Just use tabs. > Plus line is longer than 80 chars. Thanks for the catches - fixed those up and committed in c2e827ad5340fcf1735df6c77cb0311e56b985ef. Also refactored some of the xfails along the lines of what Pedro suggested in the one test case that had several similar failures (gdb.base/label.exp). If/when we fix this in Clang it might be worth refactoring into a common function (though I'm personally not very vested in keeping the test suite usable with anything other than ToT Clang - perhaps others are).