[PATCH] D41272: Don't try to run MCJIT/OrcJIT EH tests when C++ library is statically linked

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 5 14:22:43 PST 2018


phosek added a comment.

In https://reviews.llvm.org/D41272#968768, @beanz wrote:

> This looks good to me. Please correct me if I'm wrong, but it also looks like you've written this patch so that it will work correctly even if it is landed before the llvm-readobj changes are landed. If that is the case go ahead and land this.


It'd disable the MCJIT tests though because the `cxx-shared-library` wouldn't be available which is probably not something we want. However, https://reviews.llvm.org/D41527 is ready to land unless you have some other comments and https://reviews.llvm.org/D41529 has already landed so this could land immediately after.


https://reviews.llvm.org/D41272





More information about the llvm-commits mailing list