[PATCH] D88430: [OpenMP] Replace OpenMP RTL Functions With OMPIRBuilder and OMPKinds.def

Mon Sep 28 12:19:35 PDT 2020

jhuber6 added inline comments.

================
Comment at: clang/lib/CodeGen/CGOpenMPRuntimeGPU.h:38
+  /// An OpenMP-IR-Builder instance.
+  llvm::OpenMPIRBuilder OMPBuilder;
+
----------------
sstefan1 wrote:
> There's already an instance of `OpenMPIRBuilder` in `CGOpenMPRuntime`.
> 
> Maybe just make it protected?
When it's generating `__kmpc_shuffle_intxx`  around line 2480 it gets an instance of `CGOpenMPRuntimeGPU` indirectly which meant I had to make it public so it could use it. I could probably rewrite all the functions to be inside the class but I'm not sure if that'll break anything. So I just made it public for now.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88430