diff options
author | Ben Boeckel <ben.boeckel@kitware.com> | 2014-02-08 03:37:54 (GMT) |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2014-05-08 17:24:49 (GMT) |
commit | bc3856586388a610ef3d06d51c212a9ed25856c4 (patch) | |
tree | 8831a5a62c03baae846e3065b2e0414837fefed8 /Tests/RunCMake/Syntax | |
parent | 25102efc1db1e0d165ebe5e4eaef33b8206d9d3e (diff) | |
download | CMake-bc3856586388a610ef3d06d51c212a9ed25856c4.zip CMake-bc3856586388a610ef3d06d51c212a9ed25856c4.tar.gz CMake-bc3856586388a610ef3d06d51c212a9ed25856c4.tar.bz2 |
EVIS: Reimplement using custom parsing code
Introduce a new implementation of ExpandVariablesInString and select
between the old and new implementations based on policy CMP0053.
Instead of cmCommandArgumentParserHelper, use a custom parser with our
own stack. This is much faster and works well for our simple grammar.
The new behavior of CMP0053 should expand @VAR@ syntax only in certain
contexts. All existing EVIS callers use "replaceAt == true" so
hard-code our call to the old implementation. Update the signature to
default to "replaceAt == false" and pass "replaceAt == true" explicitly
in the call sites for configure_file and string(CONFIGURE).
Testing the configure (no generate) step with ParaView shows ~20%
performance improvement.
In terms of complete configure/generate steps, further testing with
ParaView shows a 20% performance improvement over 2.8.12.2 with Unix
Makefiles and minimal with Ninja. Ninja is less because it generate step
is the expensive part (future work will address this) by a long shot and
these changes help the configure step for the most part.
Diffstat (limited to 'Tests/RunCMake/Syntax')
0 files changed, 0 insertions, 0 deletions