Re: r294862 - Hopefully fixes a compile error introduced by r294861.

2017-02-12 Thread Renato Golin via cfe-commits 
On 12 February 2017 at 19:24, Aaron Ballman  wrote:
> Did, just that the test also needs a triple. I guess I'll do the dance
> to add it and re-commit.

Makes sense. Thanks!
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r294862 - Hopefully fixes a compile error introduced by r294861.

2017-02-12 Thread Aaron Ballman via cfe-commits 
On Sun, Feb 12, 2017 at 2:20 PM, Renato Golin  wrote:
> On 11 February 2017 at 18:00, Aaron Ballman via cfe-commits
>  wrote:
>> Author: aaronballman
>> Date: Sat Feb 11 12:00:32 2017
>> New Revision: 294862
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=294862=rev
>> Log:
>> Hopefully fixes a compile error introduced by r294861.
>
> Didn't... :)
>
> http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15/builds/3873 (and 
> others)
>
> Reverted both in r294910.

Did, just that the test also needs a triple. I guess I'll do the dance
to add it and re-commit.

~Aaron
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: r294862 - Hopefully fixes a compile error introduced by r294861.

2017-02-12 Thread Renato Golin via cfe-commits 
On 11 February 2017 at 18:00, Aaron Ballman via cfe-commits
 wrote:
> Author: aaronballman
> Date: Sat Feb 11 12:00:32 2017
> New Revision: 294862
>
> URL: http://llvm.org/viewvc/llvm-project?rev=294862=rev
> Log:
> Hopefully fixes a compile error introduced by r294861.

Didn't... :)

http://lab.llvm.org:8011/builders/clang-cmake-armv7-a15/builds/3873 (and others)

Reverted both in r294910.

--renato
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

r294862 - Hopefully fixes a compile error introduced by r294861.

2017-02-11 Thread Aaron Ballman via cfe-commits 
Author: aaronballman
Date: Sat Feb 11 12:00:32 2017
New Revision: 294862

URL: http://llvm.org/viewvc/llvm-project?rev=294862=rev
Log:
Hopefully fixes a compile error introduced by r294861.

Modified:
cfe/trunk/include/clang/AST/TypeLoc.h

Modified: cfe/trunk/include/clang/AST/TypeLoc.h
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/TypeLoc.h?rev=294862=294861=294862=diff
==
--- cfe/trunk/include/clang/AST/TypeLoc.h (original)
+++ cfe/trunk/include/clang/AST/TypeLoc.h Sat Feb 11 12:00:32 2017
@@ -75,22 +75,7 @@ public:
   /// adjustments from a type that wad written as a T to another type that is
   /// still canonically a T (ignores parens, attributes, elaborated types, 
etc).
   template 
-  T getAsAdjusted() const {
-TypeLoc Cur = *this;
-while (!T::isKind(Cur)) {
-  if (auto PTL = Cur.getAs())
-Cur = PTL.getInnerLoc();
-  else if (auto ATL = Cur.getAs())
-Cur = ATL.getModifiedLoc();
-  else if (auto ETL = Cur.getAs())
-Cur = ETL.getNamedTypeLoc();
-  else if (auto ATL = Cur.getAs())
-Cur = ATL.getOriginalLoc();
-  else
-break;
-}
-return Cur.getAs();
-  }
+  T getAsAdjusted() const;
 
   /// The kinds of TypeLocs.  Equivalent to the Type::TypeClass enum,
   /// except it also defines a Qualified enum that corresponds to the
@@ -2210,6 +2195,24 @@ public:
 
   QualType getInnerType() const { return this->getTypePtr()->getElementType(); 
}
 };
+
+template 
+inline T TypeLoc::getAsAdjusted() const {
+  TypeLoc Cur = *this;
+  while (!T::isKind(Cur)) {
+if (auto PTL = Cur.getAs())
+  Cur = PTL.getInnerLoc();
+else if (auto ATL = Cur.getAs())
+  Cur = ATL.getModifiedLoc();
+else if (auto ETL = Cur.getAs())
+  Cur = ETL.getNamedTypeLoc();
+else if (auto ATL = Cur.getAs())
+  Cur = ATL.getOriginalLoc();
+else
+  break;
+  }
+  return Cur.getAs();
+}
 }
 
 #endif


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