summaryrefslogtreecommitdiffstats
path: root/clang/lib/CodeGen/CGBlocks.cpp
diff options
context:
space:
mode:
authorEric Fiselier <eric@efcs.ca>2016-11-03 02:13:17 +0000
committerEric Fiselier <eric@efcs.ca>2016-11-03 02:13:17 +0000
commite4e9e281a1fbc915b6567a003fd21cc5492cde65 (patch)
treed4141b30616dd31be8d6803014f23bda7ce9ed56 /clang/lib/CodeGen/CGBlocks.cpp
parent0515fb8d4b2dbfce4fc960abb08a8d9d2f48fe2b (diff)
downloadbcm5719-llvm-e4e9e281a1fbc915b6567a003fd21cc5492cde65.tar.gz
bcm5719-llvm-e4e9e281a1fbc915b6567a003fd21cc5492cde65.zip
[Sema] Allow static_cast<T&&>(e) to check explicit conversions for non-reference-related types.
Summary: [expr.cast.static] states: > 3. A glvalue of type “cv1 T1” can be cast to type “rvalue reference to cv2 T2” if “cv2 T2” is reference-compatible > with “cv1 T1”. The result refers to the object or the specified base class subobject thereof. If T2 is > an inaccessible or ambiguous base class of T1, a program that necessitates such a cast is > ill-formed. > > 4. Otherwise, an expression e can be explicitly converted to a type T using a static_cast of the form static_- > cast<T>(e) if the declaration T t(e); is well-formed, for some invented temporary variable t. [...] Currently when checking p3 Clang will diagnose `static_cast<T&&>(e)` as invalid if the argument is not reference compatible with `T`. However I believe the correct behavior is to also check p4 in those cases. For example: ``` double y = 42; static_cast<int&&>(y); // this should be OK. 'int&& t(y)' is well formed ``` Note that we still don't check p4 for non-reference-compatible types which are reference-related since `T&& t(e);` should never be well formed in those cases. Reviewers: rsmith Subscribers: cfe-commits Differential Revision: https://reviews.llvm.org/D26231 llvm-svn: 285872
Diffstat (limited to 'clang/lib/CodeGen/CGBlocks.cpp')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud