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

Andrzej Warzynski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 11 07:32:17 PST 2022


awarzynski added a comment.

In D117016#3234228 <https://reviews.llvm.org/D117016#3234228>, @kiranchandramohan wrote:

>> 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?

Ah, I was looking for "OpenMP.h.inc" rather than "OMP.inc", so didn't find that. Both files are generated by the omp_gen <https://github.com/llvm/llvm-project/blob/main/llvm/include/llvm/Frontend/OpenMP/CMakeLists.txt> target. It does look identical to acc_gen <https://github.com/llvm/llvm-project/blob/main/llvm/include/llvm/Frontend/OpenACC/CMakeLists.txt>, so I should probably add it as a dependency too. Any thoughts?


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