[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