[PATCH] D28298: [OpenMP] Add fields for flags in the offload entry descriptor.

[Jonas Hahnfeld via Phabricator via cfe-commits]

Hahnfeld added inline comments.


================
Comment at: lib/CodeGen/CGOpenMPRuntime.h:250-252
+      // \brief Flags associated the device global.
+      int32_t Flags;
+
----------------
sfantao wrote:
> Hahnfeld wrote:
> > Is that intentionally not in the `protected` section below?
> Yes, we have a public getter/setter to interact with that, so no need to make it protected.  
Hm, `Order` and `Kind` also have public getters and are currently never set from outside - will that be required in the future?


https://reviews.llvm.org/D28298