[PATCH] D121077: Simplify OpenMP Lambda use
David Blaikie via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Mar 6 13:42:31 PST 2022
dblaikie created this revision.
dblaikie added a reviewer: ABataev.
Herald added subscribers: guansong, yaxunl.
Herald added a project: All.
dblaikie requested review of this revision.
Herald added a reviewer: jdoerfert.
Herald added subscribers: cfe-commits, sstefan1.
Herald added a project: clang.
- Use default ref capture for non-escaping lambdas (this makes maintenance easier by allowing new uses, removing uses, having conditional uses (such as in assertions) not require updates to an explicit capture list)
- Simplify addPrivate API not to take a lambda, since it calls it unconditionally/immediately anyway - most callers are simply passing in a named value or short expression anyway and the lambda syntax just adds noise/overhead
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D121077
Files:
clang/lib/CodeGen/CGOpenMPRuntime.cpp
clang/lib/CodeGen/CGStmtOpenMP.cpp
clang/lib/CodeGen/CodeGenFunction.h
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D121077.413310.patch
Type: text/x-patch
Size: 37694 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220306/e9b11478/attachment-0001.bin>
More information about the cfe-commits
mailing list