[PATCH] D117016: [flang][examples] Add missing CMake dependencies

Kiran Chandramohan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 11 06:29:06 PST 2022


kiranchandramohan added a comment.

In D117016#3234219 <https://reviews.llvm.org/D117016#3234219>, @awarzynski wrote:

> In D117016#3234201 <https://reviews.llvm.org/D117016#3234201>, @kiranchandramohan wrote:
>
>> @awarzynski  What about `omp_gen`? Do you know why it is not explicitly required?
>
> The dependency on `acc_gen` comes from this include: https://github.com/llvm/llvm-project/blob/main/flang/include/flang/Parser/parse-tree.h#L28
>
>   #include "llvm/Frontend/OpenACC/ACC.h.inc"
>
> There are no TableGen-generated OpenMP headers here. Are there any in other places? I might have missed something!

I don't remember the difference. Usually, OpenMP and OpenACC have similar handling. Anyway, further below there are includes for `ACC.inc` and `OMP.inc`. Are these includes not affected?


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