[PATCH] D28467: [Sema] Add warning for unused lambda captures
Alex Lorenz via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jan 10 06:27:43 PST 2017
arphaman added inline comments.
Comment at: include/clang/Sema/ScopeInfo.h:457
+ /// lambda.
+ bool Used = false;
> Should this be moved into one of the `PointerIntPair`s?
I'm not sure.. If we needed other capturing information in the future I would say that this should be its own field, but I'm not aware of anything else that's needed for this class. So I guess it would be better to pack it into `VarAndNestedAndThis`, but I wouldn't say it's a deal breaker.
More information about the cfe-commits