Index: lib/CodeGen/CGDebugInfo.cpp =================================================================== --- lib/CodeGen/CGDebugInfo.cpp +++ lib/CodeGen/CGDebugInfo.cpp @@ -2487,8 +2487,11 @@ SmallVector Enumerators; ED = ED->getDefinition(); for (const auto *Enum : ED->enumerators()) { - Enumerators.push_back(DBuilder.createEnumerator( - Enum->getName(), Enum->getInitVal().getSExtValue())); + const auto &InitVal = Enum->getInitVal(); + bool IsSigned = ED->getIntegerType()->isSignedIntegerType(); + auto Value = IsSigned ? InitVal.getSExtValue() : InitVal.getZExtValue(); + Enumerators.push_back( + DBuilder.createEnumerator(Enum->getName(), Value, IsSigned)); } // Return a CompositeType for the enum itself. @@ -2497,11 +2500,10 @@ llvm::DIFile *DefUnit = getOrCreateFile(ED->getLocation()); unsigned Line = getLineNumber(ED->getLocation()); llvm::DIScope *EnumContext = getDeclContextDescriptor(ED); - llvm::DIType *ClassTy = - ED->isFixed() ? getOrCreateType(ED->getIntegerType(), DefUnit) : nullptr; + llvm::DIType *ClassTy = getOrCreateType(ED->getIntegerType(), DefUnit); return DBuilder.createEnumerationType(EnumContext, ED->getName(), DefUnit, Line, Size, Align, EltArray, ClassTy, - FullName); + FullName, ED->isFixed()); } llvm::DIMacro *CGDebugInfo::CreateMacro(llvm::DIMacroFile *Parent, Index: test/CodeGen/debug-info-enum.cpp =================================================================== --- /dev/null +++ test/CodeGen/debug-info-enum.cpp @@ -0,0 +1,68 @@ +// RUN: %clang -target x86_64-linux -g -S -emit-llvm -o - %s | FileCheck %s +enum class E0 : signed char { + A0 = -128, + B0 = 127, +} x0; +// CHECK: !DICompositeType(tag: DW_TAG_enumeration_type, name: "E0" +// CHECK-SAME: flags: DIFlagFixedEnum +// CHECK: !DIBasicType(name: "signed char", size: 8, encoding: DW_ATE_signed_char) +// CHECK: !DIEnumerator(name: "A0", value: -128) +// CHECK: !DIEnumerator(name: "B0", value: 127) + +enum class E1 : unsigned char { A1 = 255 } x1; +// CHECK: !DICompositeType(tag: DW_TAG_enumeration_type, name: "E1" +// CHECK-SAME: flags: DIFlagFixedEnum +// CHECK: !DIBasicType(name: "unsigned char", size: 8, encoding: DW_ATE_unsigned_char) +// CHECK: !DIEnumerator(name: "A1", value: 255, isSigned: false) + +enum class E2 : signed short { + A2 = -32768, + B2 = 32767, +} x2; +// CHECK: !DICompositeType(tag: DW_TAG_enumeration_type, name: "E2" +// CHECK-SAME: flags: DIFlagFixedEnum +// CHECK: !DIBasicType(name: "short", size: 16, encoding: DW_ATE_signed) +// CHECK: !DIEnumerator(name: "A2", value: -32768) +// CHECK: !DIEnumerator(name: "B2", value: 32767) + +enum class E3 : unsigned short { A3 = 65535 } x3; +// CHECK: !DICompositeType(tag: DW_TAG_enumeration_type, name: "E3" +// CHECK-SAME: DIFlagFixedEnum +// CHECK: !DIBasicType(name: "unsigned short", size: 16, encoding: DW_ATE_unsigned) +// CHECK: !DIEnumerator(name: "A3", value: 65535, isSigned: false) + +enum class E4 : signed int { A4 = -2147483648, B4 = 2147483647 } x4; +// CHECK: !DICompositeType(tag: DW_TAG_enumeration_type, name: "E4" +// CHECK-SAME: flags: DIFlagFixedEnum +// CHECK: ![[INT:[0-9]+]] = !DIBasicType(name: "int", size: 32, encoding: DW_ATE_signed) +// CHECK: !DIEnumerator(name: "A4", value: -2147483648) +// CHECK: !DIEnumerator(name: "B4", value: 2147483647) + +enum class E5 : unsigned int { A5 = 4294967295 } x5; +// CHECK: !DICompositeType(tag: DW_TAG_enumeration_type, name: "E5" +// CHECK-SAME: flags: DIFlagFixedEnum +// CHECK: !DIBasicType(name: "unsigned int", size: 32, encoding: DW_ATE_unsigned) +// CHECK: !DIEnumerator(name: "A5", value: 4294967295, isSigned: false) + +enum class E6 : signed long long { + A6 = -9223372036854775807LL - 1, + B6 = 9223372036854775807LL +} x6; +// CHECK: !DICompositeType(tag: DW_TAG_enumeration_type, name: "E6" +// CHECK-SAME: DIFlagFixedEnum +// CHECK: !DIBasicType(name: "long long int", size: 64, encoding: DW_ATE_signed) +// CHECK: !DIEnumerator(name: "A6", value: -9223372036854775808) +// CHECK: !DIEnumerator(name: "B6", value: 9223372036854775807) + +enum class E7 : unsigned long long { A7 = 18446744073709551615ULL } x7; +// CHECK: !DICompositeType(tag: DW_TAG_enumeration_type, name: "E7" +// CHECK-SAME: flags: DIFlagFixedEnum +// CHECK: !DIBasicType(name: "long long unsigned int", size: 64, encoding: DW_ATE_unsigned) +// CHECK: !DIEnumerator(name: "A7", value: 18446744073709551615, isSigned: false) + +enum E8 { A8 = -128, B8 = 127 } x8; +// CHECK: !DICompositeType(tag: DW_TAG_enumeration_type, name: "E8" +// CHECK-SAME: baseType: ![[INT]] +// CHECK-NOT: flags: DIFlagFixedEnum +// CHECK: !DIEnumerator(name: "A8", value: -128) + Index: test/CodeGenCXX/debug-info-enum-class.cpp =================================================================== --- test/CodeGenCXX/debug-info-enum-class.cpp +++ test/CodeGenCXX/debug-info-enum-class.cpp @@ -15,7 +15,7 @@ // CHECK-SAME: baseType: ![[INT:[0-9]+]] // CHECK-SAME: size: 32 // CHECK-NOT: offset: -// CHECK-NOT: flags: +// CHECK-SAME: flags: DIFlagFixedEnum // CHECK-SAME: ){{$}} // CHECK: ![[INT]] = !DIBasicType(name: "int" // CHECK: !DICompositeType(tag: DW_TAG_enumeration_type, name: "B" @@ -23,12 +23,12 @@ // CHECK-SAME: baseType: ![[ULONG:[0-9]+]] // CHECK-SAME: size: 64 // CHECK-NOT: offset: -// CHECK-NOT: flags: +// CHECK-SAME: flags: DIFlagFixedEnum // CHECK-SAME: ){{$}} // CHECK: ![[ULONG]] = !DIBasicType(name: "long unsigned int" // CHECK: !DICompositeType(tag: DW_TAG_enumeration_type, name: "C" // CHECK-SAME: line: 5 -// CHECK-NOT: baseType: +// CHECK-SAME: baseType: ![[ULONG:[0-9]+]] // CHECK-SAME: size: 32 // CHECK-NOT: offset: // CHECK-NOT: flags: Index: test/CodeGenCXX/debug-info-enum.cpp =================================================================== --- test/CodeGenCXX/debug-info-enum.cpp +++ test/CodeGenCXX/debug-info-enum.cpp @@ -11,7 +11,7 @@ // CHECK-SAME: identifier: "_ZTSN5test11eE" // CHECK: [[TEST1]] = !DINamespace(name: "test1" // CHECK: [[TEST1_ENUMS]] = !{[[TEST1_E:![0-9]*]]} -// CHECK: [[TEST1_E]] = !DIEnumerator(name: "E", value: 0) +// CHECK: [[TEST1_E]] = !DIEnumerator(name: "E", value: 0, isSigned: false) enum e { E }; void foo() { int v = E; Index: test/Modules/ModuleDebugInfo.cpp =================================================================== --- test/Modules/ModuleDebugInfo.cpp +++ test/Modules/ModuleDebugInfo.cpp @@ -48,7 +48,7 @@ // CHECK: !DICompositeType(tag: DW_TAG_enumeration_type, // CHECK-NOT: name: // CHECK-SAME: ) -// CHECK: !DIEnumerator(name: "e5", value: 5) +// CHECK: !DIEnumerator(name: "e5", value: 5, isSigned: false) // CHECK: !DIDerivedType(tag: DW_TAG_typedef, name: "B", // no mangled name here yet.