[PATCH] D56113: [OpenMP] Replace predetermined shared for const variable
Alexey Bataev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 4 06:33:22 PST 2019
ABataev added inline comments.
================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:9736
+static bool rejectConstNotMutableType(Sema &SemaRef, ValueDecl *D,
+ OpenMPClauseKind CKind,
----------------
ABataev wrote:
> This function and the original code has a lot of common code. Could you, please, use this function somehow for the original too? Or outline the common code into standalone function
Also, probably missed the check if the OpenMP version is >= 4.0
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56113/new/
https://reviews.llvm.org/D56113
More information about the cfe-commits
mailing list