Prince781 added inline comments.
================
Comment at: clang/lib/Sema/SemaExpr.cpp:15329
+ if (getLangOpts().OpenMP && getLangOpts().OpenMPUseTLS) {
+ // Avoid capturing TLS-backed threadprivate variables in outer scopes.
----------------
ABataev wrote:
> this is not the right place to fix this bug, it must be fixed in
> SemaOpenMP.cpp. If you want, I can try to fix it.
I see. Thanks for your comment. I noticed elsewhere in this function
`tryCaptureVariable` that there is a `if (getLangOpts().OpenMP) ...`, so I
thought it would be okay.
If you don't mind, I wish to take another look at this and get back to you
later.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64585/new/
https://reviews.llvm.org/D64585
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
