Revert r78424.
authorArgyrios Kyrtzidis <akyrtzi@gmail.com>
Fri, 14 Aug 2009 19:01:37 +0000 (19:01 +0000)
committerArgyrios Kyrtzidis <akyrtzi@gmail.com>
Fri, 14 Aug 2009 19:01:37 +0000 (19:01 +0000)
In order for the changes in r78424 to work properly, cast_retty<X,Y> should return an object instead of a reference, and it's not clear that this approach has real advantages.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@79023 91177308-0d34-0410-b5e6-96231b3b80d8

include/llvm/Support/Casting.h

index 353611fc121e756d3bdb23097b0ee5e88de6196b..48988f8a6bb8354c71326e634364acbb2895b4b2 100644 (file)
@@ -205,9 +205,8 @@ inline typename cast_retty<X, Y>::ret_type cast(const Y &Val) {
 // accepted.
 //
 template <class X, class Y>
-inline typename cast_retty<X, Y>::ret_type cast_or_null(const Y &Val) {
-  typedef typename cast_retty<X, Y>::ret_type ret_type;
-  if (!Val) return ret_type();
+inline typename cast_retty<X, Y*>::ret_type cast_or_null(Y *Val) {
+  if (Val == 0) return 0;
   assert(isa<X>(Val) && "cast_or_null<Ty>() argument of incompatible type!");
   return cast<X>(Val);
 }
@@ -223,8 +222,7 @@ inline typename cast_retty<X, Y>::ret_type cast_or_null(const Y &Val) {
 
 template <class X, class Y>
 inline typename cast_retty<X, Y>::ret_type dyn_cast(const Y &Val) {
-  typedef typename cast_retty<X, Y>::ret_type ret_type;
-  return isa<X>(Val) ? cast<X, Y>(Val) : (ret_type)ret_type();
+  return isa<X>(Val) ? cast<X, Y>(Val) : 0;
 }
 
 // dyn_cast_or_null<X> - Functionally identical to dyn_cast, except that a null
@@ -232,8 +230,7 @@ inline typename cast_retty<X, Y>::ret_type dyn_cast(const Y &Val) {
 //
 template <class X, class Y>
 inline typename cast_retty<X, Y>::ret_type dyn_cast_or_null(const Y &Val) {
-  typedef typename cast_retty<X, Y>::ret_type ret_type;
-  return (Val && isa<X>(Val)) ? cast<X, Y>(Val) : (ret_type)ret_type();
+  return (Val && isa<X>(Val)) ? cast<X, Y>(Val) : 0;
 }