Switch from autogenerating tests to using the preprocessor.
authorChandler Carruth <chandlerc@gmail.com>
Thu, 7 Mar 2013 10:09:47 +0000 (10:09 +0000)
committerChandler Carruth <chandlerc@gmail.com>
Thu, 7 Mar 2013 10:09:47 +0000 (10:09 +0000)
commit074a3568fca118fd59fdc8d1729580d5ad295654
tree50caaa5321d5edd362ce8699bf2e901edf5e7285
parent99ee0f4790654e6df4609f5b9895db0c37dcefa6
Switch from autogenerating tests to using the preprocessor.

NOTE: You may need to run 'make clean' or 'ninja -t clean' etc!!! This
      is due to really nasty bug/interactions between
      CMake/configure/make/Ninja/LIT...

This commit tries to back out the support for generating test cases as
part of the build system due to the issues I brought up in post-commit
review:

1) It adds a *lot* of complexity and fragility to the build system. See
   the number of commits required to try to get all the bots happy.
2) It isn't really necessary -- we can already run scripts to generate
   things with the RUN lines of a test.
3) It makes the tests somewhat harder to debug as they cross between
   more domains.
4) In almost all cases it isn't really needed or it can be done directly
   using the preprocessor.

I should have been more proactive reviewing this, and I'm really sorry
about the churn here. =/ To help keep track of what commits are going
where, this backs out most of the non-test-changes from these revisions:
  r176397
  r176373
  r176293
  r176184
  r175744
  r175624
  r175545
  r175544

There were several trivial or cleanup changes to the lit files or other
files. Some of these looked ok, but I didn't try to tease them apart...
Edwin, if you know what to look for, please carry on with the cleanups
there, and sorry for hosing stuff here but I'm not much of a Python
person, and so I was erring on the side of cautiously backing out the
change.

I've tried to preserve the test changes everywhere I could, but review
is appreciated here in case I missed some.

I then re-wrote the tests to use the preprocessor rather than python to
expand to the various bits of code. The nicest part of this is that now
all the files are just C++ code. They edit and behave like C++ code,
etc. RUN lines with different -D flags are used to run the same test
over multiple different configurations, and includes bracketed in
special defines are used to flesh out a collection of standard interface
stubs to test interactions between pieces. These probably aren't perfect
yet, but I think its an improvement (at least in terms of build system
complexity) and will hopefully be a useful demonstration of the
technique I prefer for these types of tests.

llvm-svn: 176627
13 files changed:
clang-tools-extra/Makefile
clang-tools-extra/test/CMakeLists.txt
clang-tools-extra/test/Makefile
clang-tools-extra/test/cpp11-migrate/CMakeLists.txt [deleted file]
clang-tools-extra/test/cpp11-migrate/Makefile [deleted file]
clang-tools-extra/test/cpp11-migrate/UseAuto/Inputs/gen_my_std.h.py [deleted file]
clang-tools-extra/test/cpp11-migrate/UseAuto/Inputs/test_std_container.h [new file with mode: 0644]
clang-tools-extra/test/cpp11-migrate/UseAuto/basic_iterator_tests.cpp [new file with mode: 0644]
clang-tools-extra/test/cpp11-migrate/UseAuto/gen_basic_std_iterator_tests.cpp.py [deleted file]
clang-tools-extra/test/cpp11-migrate/UseAuto/iterator.cpp
clang-tools-extra/test/cpp11-migrate/lit.site.cfg.in [deleted file]
clang-tools-extra/test/lit.cfg
clang-tools-extra/test/lit.site.cfg.in