[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