Hi David,
On Mon, Jun 13, 2022 at 2:00 PM David Blaikie <dblai...@gmail.com> wrote:
> On Sun, Jun 12, 2022 at 10:17 AM Kazu Hirata via cfe-commits <
> cfe-commits@lists.llvm.org> wrote:
>
>> - return args_end() != std::find_if(
>> - args_begin(), args_end(),
>> - [=](const ParamIdx &Idx) { return Idx.getASTIndex() == IdxAST;
>> });
>> + return llvm::any_of(args(), [=](const ParamIdx &Idx) {
>> + return Idx.getASTIndex() == IdxAST;
>> + });
>>
>
> Generally, I think for locally scoped lambdas (lambdas that don't escape
> their scope) we should prefer/default to default reference capture (`[&]`)
> - it simplifies changes in the future. Would that be OK for these sort of
> cleanups?
>
Sure. I've often wondered about the point of explicitly specifying
captures.
Kazu Hirata
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
