Commit Graph

20 Commits

Author SHA1 Message Date
Zachary Turner
79708b54f2 Re-revert "Refactor debuginfo-tests."
This is still breaking greendragon.

At this point I give up until someone can fix the greendragon
bots, and I will probably abandon this effort in favor of using
a private github repository.

llvm-svn: 318722
2017-11-21 01:20:28 +00:00
Zachary Turner
8b6ef88e7e Resubmit "Refactor debuginfo-tests" again.
This was reverted due to the tests being run twice on some
build bots.  Each run had a slightly different configuration
due to the way in which it was being invoked.  This fixes
the problem (albeit in a somewhat hacky way).  Hopefully in
the future we can get rid of the workflow of running
debuginfo-tests as part of clang, and then this hack can
go away.

llvm-svn: 318697
2017-11-20 21:41:36 +00:00
Zachary Turner
b5c237ec3d Re-revert "Refactor debuginfo-tests"
This is still broken because it causes certain tests to be
run twice with slightly different configurations, which is
wrong in some cases.

You can observe this by running:

  ninja -nv check-all | grep debuginfo-tests

And seeing that it passes clang/test and clang/test/debuginfo-tests
to lit, which causes it to run debuginfo-tests twice.  The fix is
going to involve either:

  a) figuring out that we're running in this "deprecated" configuration,
     and then deleting the clang/test/debuginfo-tests path, which should
     cause it to behave identically to before, or:
  b) make lit smart enough that it doesn't descend into a sub-suite if
     that sub-suite already has a lit.cfg file.

llvm-svn: 318486
2017-11-17 00:41:18 +00:00
Zachary Turner
5e420717a1 Resubmit "Refactor debuginfo-tests"
This was reverted due to some failures on specific darwin buildbots,
the issue being that the new lit configuration was not setting the
SDKROOT environment variable.  We've tested a fix locally and confirmed
that it works, so this patch resubmits everything with the fix
applied.

llvm-svn: 318435
2017-11-16 18:26:20 +00:00
Zachary Turner
faf04a09f6 Revert "Update test_debuginfo.pl script to point to new tree location."
This reverts the aforementioned patch and 2 subsequent follow-ups,
as some buildbots are still failing 2 tests because of it.
Investigation is ongoing into the cause of the failures.

llvm-svn: 318112
2017-11-13 23:33:29 +00:00
Zachary Turner
0f2ce11df7 [debuginfo-tests] Make debuginfo-tests work in a standard configuration.
Previously, debuginfo-tests was expected to be checked out into
clang/test and then the tests would automatically run as part of
check-clang.  This is not a standard workflow for handling
external projects, and it brings with it some serious drawbacks
such as the inability to depend on things other than clang, which
we will need going forward.

The goal of this patch is to migrate towards a more standard
workflow.  To ease the transition for build bot maintainers,
this patch tries not to break the existing workflow, but instead
simply deprecate it to give maintainers a chance to update
the build infrastructure.

Differential Revision: https://reviews.llvm.org/D39605

llvm-svn: 317925
2017-11-10 20:57:57 +00:00
Filipe Cabecinhas
090a237f29 New round of fixes for "Always compile debuginfo-tests for the host triple"
clang tests were breaking, at least when compiling clang only, from an
installed llvm. Make the lit.cfg script deal with the case when we don't
have a host_triple available.

llvm-svn: 220152
2014-10-18 23:47:59 +00:00
Filipe Cabecinhas
63f2c2cb65 Always compile debuginfo-tests for the host triple
Since these tests expect a working debugger, always compile them for the
host triple, assuming a working debugger is present.

This enables us to compile and run them, even when clang is, by default,
a cross-compiler (but can still target the host).

llvm-svn: 219992
2014-10-16 23:43:34 +00:00
Adrian Prantl
cece9ccaf1 Revert "Experimentally enable --verify-debug-info on some testcases and see if"
Some of the buildbots need to be upgraded to a more recen version of
dwarfdump first.

Reverting for now.

llvm-svn: 203338
2014-03-08 02:46:36 +00:00
Adrian Prantl
f3c43d14d9 Experimentally enable --verify-debug-info on some testcases and see if
the buildbots can take it.

llvm-svn: 203327
2014-03-08 01:23:49 +00:00
Adrian Prantl
11f0d2ce92 Relax testcase: Newer versions of LLDB don't necessarily print the "{" either.
llvm-svn: 202319
2014-02-27 00:02:40 +00:00
Adrian Prantl
d913c86b93 don't use CHECK-NEXT because it may be on the same line.
llvm-svn: 190969
2013-09-18 23:01:54 +00:00
Adrian Prantl
7f6f3bb86b Massage those regexps to work with both lldb and gdb.
llvm-svn: 190967
2013-09-18 22:56:35 +00:00
Adrian Prantl
888e023e0d Add an lldb wrapper script that implements gdb-compatible commands on top
of lldb. This will be the new default method for executing the tests in
this repository on Darwin.
There is also a related patch for llvm/utils/test_debuginfo.pl coming.

I also relaxed some of the checks to work with both gdb's and lldb's
output.

llvm-svn: 190185
2013-09-06 18:10:44 +00:00
David Blaikie
86feadcc76 Now that we get non-trivial value parameters locations correct, un-XFAIL these tests.
llvm-svn: 184372
2013-06-19 22:07:11 +00:00
David Blaikie
fbbb8f32b0 We've got the type right, which has broken these tests - but now we need to get the location information correct that the type was papering over
llvm-svn: 183576
2013-06-07 22:09:23 +00:00
Eric Christopher
e6a602dfab Revert previous patch as the corresponding clang patch was reverted.
gdb's tests are too dependent upon order in debug info.

llvm-svn: 149050
2012-01-26 07:01:33 +00:00
Eric Christopher
7b878febf1 Reverse the order that these are expected for clang change. Should
be no functional change (the offsets in AT_data_member_location are still
correct). If the gdb testsuite itself needs changing then I'll revert this
and the clang patch.

llvm-svn: 149026
2012-01-26 02:33:53 +00:00
Devang Patel
443e46f16d Silence return-type warning.
llvm-svn: 130349
2011-04-28 00:02:06 +00:00
Devang Patel
d9bc8e0817 test case for r125249.
llvm-svn: 125250
2011-02-10 00:41:14 +00:00