ABataev added inline comments. ================ Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:2991-3005 @@ +2990,17 @@ + +/// \brief Values for bit flags used to specify the mapping type for +/// offloading. +enum OpenMPOffloadMappingFlags { + /// \brief Allocate memory on the device and move data from host to device. + OMP_MAP_TO = 0x01, + /// \brief Allocate memory on the device and move data from device to host. + OMP_MAP_FROM = 0x02, +}; + +enum OpenMPOffloadingReservedDeviceIDs { + /// \brief Device ID if the device was not defined, runtime should get it + /// from environment variables in the spec. + OMP_DEVICEID_UNDEF = -1, +}; + +void CGOpenMPRuntime::emitTargetCall(CodeGenFunction &CGF, ---------------- Move them to CGOpenMPRuntime::emitTargetCall(), they can be made local
================ Comment at: test/OpenMP/target_codegen.cpp:8 @@ +7,3 @@ +// expected-no-diagnostics +// REQUIRES: powerpc-registered-target +#ifndef HEADER ---------------- Some of your tests has triple i386, they don't need PowerPC target http://reviews.llvm.org/D11361 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits