[PATCH] D29905: [OpenMP] Pass argument to device kernel by reference when map is used.

Gheorghe-Teodor Bercea via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 11 07:49:20 PDT 2017


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


================
Comment at: lib/Sema/SemaOpenMP.cpp:358-360
+  /// Do the check specified in \a Check to all component lists at a given level
+  /// and return true if any issue is found.
+  bool checkMappableExprComponentListsForDeclAtLevel(
----------------
ABataev wrote:
> Could you join these 2 functions into one?
Since SI and StarI use different types of iterations the functions cannot be merged.


Repository:
  rL LLVM

https://reviews.llvm.org/D29905





More information about the cfe-commits mailing list