On 25/09/2014 06:20, Ehsan Akhgari wrote:
http://reviews.llvm.org/D5488

It seems you are only using plain 'auto'. This will cause unnecessary copies and in cases where copies are cheap (as they are pointers) this is not obvious from looking at the code.

This link explains the issues and the right way to use auto in range based loops:

http://llvm.org/docs/CodingStandards.html#beware-unnecessary-copies-with-auto

Cheers,
Tobias
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to