[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