[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.
More information about the llvm-commits