tianshilei1992 added a comment.
In D116637#3284773 <https://reviews.llvm.org/D116637#3284773>, @ABataev wrote:
> sema and parsing tests?
Tests will be in the patch for code gen.
================
Comment at: clang/lib/Sema/SemaOpenMP.cpp:10921
+/// Get the node id of the fixed point of an expression \a S.
+llvm::FoldingSetNodeID getNodeId(ASTContext &Context, const Expr *S) {
+ llvm::FoldingSetNodeID Id;
----------------
ABataev wrote:
> static
Sure, but I usually prefer not to use `static` for a function in an anonymous
name space.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116637/new/
https://reviews.llvm.org/D116637
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
