float, double, and long double do need extra data in the

BuiltinTypeLoc structure. Thanks, Enea!

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

diff --git a/include/clang/AST/TypeLoc.h b/include/clang/AST/TypeLoc.h
index 0a40ea1fc47d6df6adfda409a932a3e6df0db644..914cc8d0f095b45dddd895babc544837fcebba7d 100644 (file)
--- a/include/clang/AST/TypeLoc.h
+++ b/include/clang/AST/TypeLoc.h
@@ -404,7 +404,7 @@ public:
   bool needsExtraLocalData() const {
     BuiltinType::Kind bk = getTypePtr()->getKind();
     return (bk >= BuiltinType::UShort && bk <= BuiltinType::UInt128)
-      || (bk >= BuiltinType::Short && bk <= BuiltinType::Int128)
+      || (bk >= BuiltinType::Short && bk <= BuiltinType::LongDouble)
       || bk == BuiltinType::UChar
       || bk == BuiltinType::SChar;
   }

diff --git a/lib/AST/TypeLoc.cpp b/lib/AST/TypeLoc.cpp
index 6da2ab0dee628109d8bd74dd6f34ecb8ba520036..fd9fbc191868a9b24598665fd3c7c71920e12683 100644 (file)
--- a/lib/AST/TypeLoc.cpp
+++ b/lib/AST/TypeLoc.cpp
@@ -156,11 +156,6 @@ TypeSpecifierType BuiltinTypeLoc::getWrittenTypeSpec() const {
       return TST_char32;
     case BuiltinType::WChar:
       return TST_wchar;
-    case BuiltinType::Float:
-      return TST_float;
-    case BuiltinType::Double:
-    case BuiltinType::LongDouble:
-      return TST_double;
     case BuiltinType::UndeducedAuto:
       return TST_auto;
         
@@ -176,6 +171,9 @@ TypeSpecifierType BuiltinTypeLoc::getWrittenTypeSpec() const {
     case BuiltinType::Long:
     case BuiltinType::LongLong:
     case BuiltinType::Int128:
+    case BuiltinType::Float:
+    case BuiltinType::Double:
+    case BuiltinType::LongDouble:
       llvm_unreachable("Builtin type needs extra local data!");
       // Fall through, if the impossible happens.