Re: [PATCH] D11928: Small fixup

2015-08-11 Thread Nathan Wilson via cfe-commits
nwilson added a subscriber: nwilson. nwilson added a comment. Hi Piotr, Would you mind renaming the title of the Patch since the title will be committed as such? Thanks! http://reviews.llvm.org/D11928 ___ cfe-commits mailing list cfe-commits@lists

Re: [PATCH] D11928: Small fixup

2015-08-11 Thread Piotr Padlewski via cfe-commits
Prazek added inline comments. Comment at: include/clang/AST/VTableBuilder.h:54 @@ -53,3 +53,3 @@ - VTableComponent() { } + VTableComponent() = default; rnk wrote: > Is this ctor used? It leaves Value uninitialized. Maybe we should delete it > to ensure that

Re: [PATCH] D11928: Small fixup

2015-08-11 Thread Reid Kleckner via cfe-commits
rnk added inline comments. Comment at: include/clang/AST/VTableBuilder.h:54 @@ -53,3 +53,3 @@ - VTableComponent() { } + VTableComponent() = default; Is this ctor used? It leaves Value uninitialized. Maybe we should delete it to ensure that it isn't called,

Re: [PATCH] D11928: Small fixup

2015-08-11 Thread Richard Smith via cfe-commits
rsmith added a comment. This change now disables the move constructor and move assignment. I think just removing the comment is better. http://reviews.llvm.org/D11928 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi

Re: [PATCH] D11928: Small fixup

2015-08-11 Thread Piotr Padlewski via cfe-commits
Prazek updated this revision to Diff 31863. http://reviews.llvm.org/D11928 Files: include/clang/AST/VTableBuilder.h Index: include/clang/AST/VTableBuilder.h === --- include/clang/AST/VTableBuilder.h +++ include/clang/AST/VTableBui

Re: [PATCH] D11928: Small fixup

2015-08-11 Thread Reid Kleckner via cfe-commits
rnk added a comment. I added this code before C++11 migration. I wanted to explicitly default the copy ctor to document that it's a value type, rather than defaulting it implicitly. You can nuke both comments and uncomment the definition. http://reviews.llvm.org/D11928 _

Re: [PATCH] D11928: Small fixup

2015-08-11 Thread Richard Smith via cfe-commits
rsmith added a comment. Reid, do you remember what this FIXME was for? (Added in r198462.) http://reviews.llvm.org/D11928 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits