erichkeane marked 2 inline comments as done.
erichkeane added a comment.
Ugg... well conversion functions are an interesting bit, as well as Type{};
with no constructors. It ends up being a function style cast to an
init-list-expr, so its going to require a bit more work. Stay tuned :)
================
Comment at: clang/lib/Sema/SemaDeclAttr.cpp:2835
+ D->getFunctionType()->getReturnType()->isVoidType() &&
+ !isa<CXXConstructorDecl>(D)) {
S.Diag(AL.getLoc(), diag::warn_attribute_void_function_method) << AL << 0;
----------------
aaron.ballman wrote:
> Conversion functions?
Conversion functions don't return 'void', so the 2nd condition should do it,
right?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D64914/new/
https://reviews.llvm.org/D64914
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
