[PATCH] D74588: Use DISABLE_LLVM_LINK_LLVM_DYLIB for TableGenTests
Aaron Puchert via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 30 09:43:23 PDT 2021
aaronpuchert added a comment.
In D74588#2658871 <https://reviews.llvm.org/D74588#2658871>, @beanz wrote:
> Yep, and that's because of a different bug in how LLVMTableGenGlobalISel was setup.
>
> This patch would address the issues: [...]
Ok, that allows me to remove the `DISABLE_LLVM_LINK_LLVM_DYLIB`, but
- maybe it would be confusing to have an LLVM component in `llvm/utils` instead of `llvm/lib`, and
- maybe this shouldn't be part of LLVM because it's only needed for `llvm-tblgen`? In other words maybe this isn't actually a component of LLVM proper but just a tool for building LLVM. If we make it a component library it will be part of `libLLVM.so`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D74588/new/
https://reviews.llvm.org/D74588
More information about the llvm-commits
mailing list