[PATCH] D117016: [flang][examples] Add missing CMake dependencies
Andrzej Warzynski via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 24 01:31:56 PST 2022
awarzynski added a comment.
Thanks for the comments, @clementval !
In D117016#3248081 <https://reviews.llvm.org/D117016#3248081>, @clementval wrote:
> I'm ok with the change.
Ta!
> The only solution would be to extract OpenMP and OpenACC specific parts to their own include files (...)
Sounds like a good idea.
> (...) but in your example you would need the `omp_gen` dependency.
Only in one of them - `FlangOmpReport`. But that makes sense - it's for finding OpenMP construct/clauses anyway. The other one, `PrintFlangFunctionNames`, doesn't need it. And neither of the examples requires OpenACC.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117016/new/
https://reviews.llvm.org/D117016
More information about the llvm-commits
mailing list