Re: [PATCH] D12134: [DEBUG INFO] Source correlation for lambda captured values.

2015-08-25 Thread David Blaikie via cfe-commits
Hmm, my mistake, it did - my mail client's just not threaded it together for some reason. :/ On Tue, Aug 25, 2015 at 8:06 AM, David Blaikie wrote: > Looks like the initial mail didn't hit the mailing list? Does someone want > to restart this review and/or forward that initial mail with the > pat

Re: [PATCH] D12134: [DEBUG INFO] Source correlation for lambda captured values.

2015-08-25 Thread David Blaikie via cfe-commits
Looks like the initial mail didn't hit the mailing list? Does someone want to restart this review and/or forward that initial mail with the patch/description/etc? On Mon, Aug 24, 2015 at 11:00 PM, Eric Christopher via cfe-commits < cfe-commits@lists.llvm.org> wrote: > echristo added a comment. >

Re: [PATCH] D12134: [DEBUG INFO] Source correlation for lambda captured values.

2015-08-25 Thread Alexey Bataev via cfe-commits
ABataev added a subscriber: ABataev. ABataev added a comment. Yes, you're right, and it will be 13, 14, 15. It is implemented already and this patch does not break this. Best regards, Alexey Bataev = Software Engineer Intel Compiler Team 25.08.2015 9:00, Eric Christopher пишет: >

Re: [PATCH] D12134: [DEBUG INFO] Source correlation for lambda captured values.

2015-08-25 Thread Bataev, Alexey via cfe-commits
Yes, you're right, and it will be 13, 14, 15. It is implemented already and this patch does not break this. Best regards, Alexey Bataev = Software Engineer Intel Compiler Team 25.08.2015 9:00, Eric Christopher пишет: echristo added a comment. How about: 13. [apple, 14. banana, 15

Re: [PATCH] D12134: [DEBUG INFO] Source correlation for lambda captured values.

2015-08-24 Thread Eric Christopher via cfe-commits
echristo added a comment. How about: 13. [apple, 14. banana, 15. cherry]{ 16. printf("apple = %d\n",apple); 17. printf("banana = %d\n",banana); 18. printf("cherry = %d\n",cherry); Should be 13, 14, 15 yes? -eric http://reviews.llvm.org/D12134 ___