[PATCH] D26115: [test] Correctly include build llvm_shlib_dir in stand-alone builds

Chris Bieneman via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 14 13:53:46 PST 2016


beanz 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)
----------------
Should this really be fatal? It seems to me in many cases you might not need this to be set.


https://reviews.llvm.org/D26115





More information about the cfe-commits mailing list