[PATCH] D37818: [lit] Update clang and lld to use the new shared LLVMConfig stuff

2017-09-13 Thread David L. Jones via Phabricator via cfe-commits
dlj added inline comments. Comment at: clang/test/lit.cfg:23 # the test runner updated. -config.test_format = lit.formats.ShTest(execute_external) +config.test_format = lit.formats.ShTest(not llvm_config.use_lit_shell) Minor nit: it seems reasonable enough to

[PATCH] D37818: [lit] Update clang and lld to use the new shared LLVMConfig stuff

2017-09-13 Thread Reid Kleckner via Phabricator via cfe-commits
rnk added inline comments. Comment at: llvm/utils/lit/lit/llvm/config.py:92 def with_environment(self, variable, value, append_path = False): +if append_path: Rather than having an optional parameter that makes this append, maybe have a new method

[PATCH] D37818: [lit] Update clang and lld to use the new shared LLVMConfig stuff

2017-09-13 Thread Zachary Turner via Phabricator via cfe-commits
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) Th