[PATCH] D26115: [test] Correctly include build llvm_shlib_dir in stand-alone builds
Michał Górny via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 14 14:01:45 PST 2016
mgorny added inline comments.
================
Comment at: test/lit.cfg:109
+ if not llvm_shlib_dir:
+ lit_config.fatal('No LLVM shlib dir set!')
llvm_libs_dir = getattr(config, 'llvm_libs_dir', None)
----------------
beanz wrote:
> Should this really be fatal? It seems to me in many cases you might not need this to be set.
I don't mind either way. I left it like this since I really don't see a case when it could be unset ;-).
https://reviews.llvm.org/D26115
More information about the cfe-commits
mailing list