[PATCH] D37818: [lit] Update clang and lld to use the new shared LLVMConfig stuff
Zachary Turner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 13 11:52:41 PDT 2017
zturner created this revision.
Herald added a reviewer: modocache.
Herald added a subscriber: fedor.sergeev.
This is probably the last work I'm going to do here for a while. I still see some room for improvement, but I think after this patch:
a) refactor begins to have diminishing returns
b) There's a sufficient amount of sharing and re-use to validate the design
c) There's enough specific examples of how to use this that other projects (e.g. compiler-rt, etc) can use those as a model of how to simplify their own configs.
In other words, I'm not brave enough to touch compiler-rt's config files :)
There's some minor changes in the config api that increase flexibility in how you override environment variables and spawn an external `llvm-config`, but otherwise this is mostly just deleting code.
https://reviews.llvm.org/D37818
Files:
clang/test/lit.cfg
clang/test/lit.site.cfg.in
lld/test/lit.cfg
lld/test/lit.site.cfg.in
llvm/test/lit.cfg
llvm/utils/lit/lit/llvm/config.py
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37818.115084.patch
Type: text/x-patch
Size: 23563 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170913/2aae288a/attachment.bin>
More information about the llvm-commits
mailing list