[PATCH] D33526: Fix spurious Wunused-lambda-capture warning
Yi Kong via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri May 26 16:45:19 PDT 2017
kongyi added inline comments.
Comment at: lib/Sema/SemaLambda.cpp:1524-1526
+ if (!CurContext->isDependentContext() && !IsImplicit)
+ if ((IsGenericLambda && !From.isNonODRUsed()) ||
+ (!IsGenericLambda && !From.isODRUsed()))
> Should generic lambdas and lambdas in a dependent context be treated the same?
> Is a lambda inside a generic lambda in a dependent context?
Generic lambdas are essentially templated lambdas. For diagnosing unused captures, they can be treated as the same. However we are not generating diagnoses within dependent contexts right now (L1524), so this is not really related to this fix.
More information about the cfe-commits