diff options
author | Bill Wendling <isanbard@gmail.com> | 2012-11-13 02:31:47 +0000 |
---|---|---|
committer | Bill Wendling <isanbard@gmail.com> | 2012-11-13 02:31:47 +0000 |
commit | e7ff4c14b157746b3e0228d2dce9f70712d1c126 (patch) | |
tree | cb2924f6d68ba921a83cd91cde117e605f232cab /lib | |
parent | f546ac5f9bdedc7d1ae49238c65a93201d0e4f05 (diff) | |
download | llvm-e7ff4c14b157746b3e0228d2dce9f70712d1c126.tar.gz llvm-e7ff4c14b157746b3e0228d2dce9f70712d1c126.tar.bz2 llvm-e7ff4c14b157746b3e0228d2dce9f70712d1c126.tar.xz |
Use the 'count' attribute instead of the 'upper_bound' attribute.
If we have a type 'int a[1]' and a type 'int b[0]', the generated DWARF is the
same for both of them because we use the 'upper_bound' attribute. Instead use
the 'count' attrbute, which gives the correct number of elements in the array.
<rdar://problem/12566646>
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@167806 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib')
-rw-r--r-- | lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp | 3 | ||||
-rw-r--r-- | lib/VMCore/DIBuilder.cpp | 6 |
2 files changed, 6 insertions, 3 deletions
diff --git a/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp b/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp index 2b07dda31f..bcb002cdac 100644 --- a/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp +++ b/lib/CodeGen/AsmPrinter/DwarfCompileUnit.cpp @@ -1252,6 +1252,7 @@ void CompileUnit::constructSubrangeDIE(DIE &Buffer, DISubrange SR, addDIEEntry(DW_Subrange, dwarf::DW_AT_type, dwarf::DW_FORM_ref4, IndexTy); uint64_t L = SR.getLo(); uint64_t H = SR.getHi(); + uint64_t C = SR.getCount(); // The L value defines the lower bounds which is typically zero for C/C++. The // H value is the upper bounds. Values are 64 bit. H - L + 1 is the size @@ -1265,7 +1266,7 @@ void CompileUnit::constructSubrangeDIE(DIE &Buffer, DISubrange SR, } if (L) addUInt(DW_Subrange, dwarf::DW_AT_lower_bound, 0, L); - addUInt(DW_Subrange, dwarf::DW_AT_upper_bound, 0, H); + addUInt(DW_Subrange, dwarf::DW_AT_count, 0, C); Buffer.addChild(DW_Subrange); } diff --git a/lib/VMCore/DIBuilder.cpp b/lib/VMCore/DIBuilder.cpp index 152b825523..ed9c3b0a36 100644 --- a/lib/VMCore/DIBuilder.cpp +++ b/lib/VMCore/DIBuilder.cpp @@ -741,11 +741,13 @@ DIArray DIBuilder::getOrCreateArray(ArrayRef<Value *> Elements) { /// getOrCreateSubrange - Create a descriptor for a value range. This /// implicitly uniques the values returned. -DISubrange DIBuilder::getOrCreateSubrange(int64_t Lo, int64_t Hi) { +DISubrange DIBuilder::getOrCreateSubrange(int64_t Lo, int64_t Hi, + uint64_t Count) { Value *Elts[] = { GetTagConstant(VMContext, dwarf::DW_TAG_subrange_type), ConstantInt::get(Type::getInt64Ty(VMContext), Lo), - ConstantInt::get(Type::getInt64Ty(VMContext), Hi) + ConstantInt::get(Type::getInt64Ty(VMContext), Hi), + ConstantInt::get(Type::getInt64Ty(VMContext), Count) }; return DISubrange(MDNode::get(VMContext, Elts)); |