koops added inline comments.

================
Comment at: clang/include/clang/Sema/Sema.h:11094
 
+  bool checkLastPrivateForMappedDirectives(ArrayRef<OMPClause *> Clauses);
+  bool mapLoopConstruct(llvm::SmallVector<OMPClause *, 8> *ClausesWithoutBind,
----------------
ABataev wrote:
> const member function?
> Add a comment
"checkLastPrivateForMappedDirectives" is calling a non-const function 
"checkGenericLoopLastprivate". Since, this is a pre-existing function and is 
being called from other locations, I cannot make 
"checkLastPrivateForMappedDirectives" a const.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144634/new/

https://reviews.llvm.org/D144634

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

Reply via email to