[PATCH] D43435: Expand a lambda that is used only once.
Sam Clegg via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 19 10:48:51 PST 2018
sbc100 accepted this revision.
sbc100 added a comment.
This revision is now accepted and ready to land.
I was considering doing that myself, and managed to persuade myself it was more readable with the lamda since it makes the loop itself self-documenting and really easy to read. But I can see that single use lambda are probably not a good pattern and that a commit is probably just as good. BTW, do you now if there is any runtime different between these two (given a decent compiler)?
https://reviews.llvm.org/D43435
More information about the llvm-commits
mailing list