[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