[PATCH] D69922: [OpenMP] Use the OpenMP-IR-Builder

[Kiran Chandramohan via Phabricator via llvm-commits]

kiranchandramohan added a comment.

Thanks @jdoerfert for working on this.

Sorry for being late to the party. I am trying to implement one trivial directive (Flush) and one slightly more involved (not decided).

I applied D69853 <https://reviews.llvm.org/D69853>, D69785 <https://reviews.llvm.org/D69785>, D69922 <https://reviews.llvm.org/D69922> to my local build and found that D69922 <https://reviews.llvm.org/D69922> is referring to OpenMPIRBuilder.h in llvm/Frontend whereas in D69785 <https://reviews.llvm.org/D69785> it was introduced in llvm/Transforms/Utils/OpenMPIRBuilder.h



================
Comment at: clang/lib/CodeGen/CGOpenMPRuntime.cpp:29
 #include "llvm/Support/raw_ostream.h"
+#include "llvm/Frontend/OpenMPIRBuilder.h"
 #include <cassert>
----------------
D69785 has this file in llvm/Transforms/Utils/OpenMPIRBuilder.h


================
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:55
 #include "llvm/IR/Module.h"
+#include "llvm/Frontend/OpenMPIRBuilder.h"
 #include "llvm/IR/ProfileSummary.h"
----------------
D69785 has this file in llvm/Transforms/Utils/OpenMPIRBuilder.h


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69922/new/

https://reviews.llvm.org/D69922