sfantao added inline comments.

================
Comment at: lib/CodeGen/CGOpenMPRuntime.h:250-252
+      // \brief Flags associated the device global.
+      int32_t Flags;
+
----------------
Hahnfeld wrote:
> 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?
Good point, I guess those fields were being directly accessed by the child 
classes but now they are set through the constructor. I am privatizing the all 
three fields in the last diff as they don't need to be protected anymore.


https://reviews.llvm.org/D28298



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to