[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