[lldb][NFC] Rewrite CPP11EnumTypes test to make it faster
authorRaphael Isemann <teemperor@gmail.com>
Fri, 4 Sep 2020 10:29:40 +0000 (12:29 +0200)
committerRaphael Isemann <teemperor@gmail.com>
Fri, 4 Sep 2020 11:45:42 +0000 (13:45 +0200)
commit101f37a1b330e3f0ae57762db47bba28f72cf50d
tree2c05fcca2be2a488a5fe363249c74256f3dfeb5e
parent2c9dbcda4f71497d4a58020bb093af438fb6e967
[lldb][NFC] Rewrite CPP11EnumTypes test to make it faster

TestCPP11EnumTypes is one of the most expensive tests on my system and takes
around 35 seconds to run. A relatively large amount of that time is actually
doing CPU intensive work it seems (and not waiting on timeouts like other
slow tests).

The main issue is that this test repeatedly compiles the same source files
with different compiler defines. The test is also including standard library
headers, so it will also build all system modules with the gmodules debug
info variant. This leads to the problem that this test ends up compiling all
system Clang modules 8 times (one for each subtest with a unique define). As
the system modules are quite large, this causes that this test spends most
of its runtime just recompiling all system modules on macOS.

There is also the small issue that this test is starting and start-stopping
the test process a few hundred times.

This rewrites the test to instead just use a macro to instantiate all the
enum types in a single source and uses global variables to test the values
(which means there is no more need to continue/stop or even start a process).

I kept running all the debug info variants (event though it doesn't seem really
relevant) to keep this as NFC as possible.

This reduced the test runtime by around 1.5 seconds on my system (or in relative
numbers, the runtime of this test decreases by 95%).
lldb/test/API/lang/cpp/enum_types/TestCPP11EnumTypes.py
lldb/test/API/lang/cpp/enum_types/main.cpp