[PATCH] D36580: [OpenCL] Support variable memory scope in atomic builtins

Yaxun Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 15 21:17:15 PDT 2017


yaxunl marked an inline comment as done.
yaxunl added inline comments.


================
Comment at: cfe/trunk/lib/Frontend/InitPreprocessor.cpp:581
   // Define macros for the OpenCL memory scope.
   // The values should match clang SyncScope enum.
+  static_assert(
----------------
t-tye wrote:
> // The values should match clang AtomicScopeOpenCLModel::ID enum.
Fixed. Thanks.


Repository:
  rL LLVM

https://reviews.llvm.org/D36580





More information about the cfe-commits mailing list