]> granicus.if.org Git - clang/commit
PR15132: Replace "address expression must be an lvalue or a function
authorRichard Smith <richard-llvm@metafoo.co.uk>
Sat, 2 Feb 2013 02:14:45 +0000 (02:14 +0000)
committerRichard Smith <richard-llvm@metafoo.co.uk>
Sat, 2 Feb 2013 02:14:45 +0000 (02:14 +0000)
commit3fa3feab35096b608f1d79bb541798b37a55e7b9
tree9d1266173a5b7da78cf0998856044dea5c9b1e30
parent821b93eec8b58a3e320ef34e7c98906ab61cf8c3
PR15132: Replace "address expression must be an lvalue or a function
designator" diagnostic with more correct and more human-friendly "cannot take
address of rvalue of type 'T'".

For the case of & &T::f, provide a custom diagnostic, rather than unhelpfully
saying "cannot take address of rvalue of type '<overloaded function type>'".

For the case of &array_temporary, treat it just like a class temporary
(including allowing it as an extension); the existing diagnostic wording
for the class temporary case works fine.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@174262 91177308-0d34-0410-b5e6-96231b3b80d8
12 files changed:
include/clang/Basic/DiagnosticSemaKinds.td
lib/Sema/SemaExpr.cpp
test/CodeGenCXX/cxx0x-initializer-array.cpp
test/Parser/objcxx0x-lambda-expressions.mm
test/Sema/complex-imag.c
test/Sema/expr-address-of.c
test/Sema/varargs.c
test/SemaCXX/address-of.cpp
test/SemaCXX/conditional-expr.cpp
test/SemaCXX/cxx0x-initializer-aggregates.cpp
test/SemaCXX/nullptr.cpp
test/SemaTemplate/temp_arg_nontype.cpp