[PATCH] D91468: [llvm] Add OpenACC as a dependency to llvm-config
Tom Stellard via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 13 16:53:46 PST 2020
tstellar added a comment.
I know this fixes a bug, but I don't think this is quite correct. It's really only the test that depends on the library not the tool. You could add LLVMFrontendOpenACC as a dependency to the check target, but it's pretty uncommon for an LLVM library to not have any tests. I think it would be better to fix this by adding a test case for LLVMFrontendOpenACC.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91468/new/
https://reviews.llvm.org/D91468
More information about the llvm-commits
mailing list