[PATCH] D11784: [PATCH] clang-tidy check for incorrect move constructor initializers

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 14 05:05:30 PDT 2015

aaron.ballman added a comment.

In http://reviews.llvm.org/D11784#220654, @aaron.ballman wrote:

> Ping?
> FWIW, this patch almost caught a bug in LLVM. ;-) DependenceAnalysis.h has a class: FullDependence which would suffer from this problem if the Dependence base class did not accidentally suppress creation of the move constructor by defaulting only the copy constructor. (Separate patch forthcoming.) I think that may be a reasonable option for this patch to test for, but wasn't quite certain. What do others think?

Do you think that this patch should have an option for the case where the initialization cannot use a move constructor because the default one is deleted?



More information about the cfe-commits mailing list