ClockMan added a comment.

Thing is that same issue may happen with all other members, what about 
copy/move constructors defaulted in .cpp (just to speed up compilation for 
classes with many members).
Best thing would simply to check if all definitions are available. In that case 
issues would be reported only for source file that corresponds to header file 
with defined class.
This would also reduce amount of generated warnings for a big project that 
include such header with class in many places.

Other good option would be to excluded classes defined in system headers, no 
point to check if some boost class or std::vector got proper constructors.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143851/new/

https://reviews.llvm.org/D143851

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to