[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