diff options
author | Eric Christopher <echristo@gmail.com> | 2013-12-20 04:34:22 +0000 |
---|---|---|
committer | Eric Christopher <echristo@gmail.com> | 2013-12-20 04:34:22 +0000 |
commit | d1946f7afe06924fdf7d3c9ad4b111aef8294ee1 (patch) | |
tree | 17d3076c2021b3064766415a95b998eb9f8b889f | |
parent | e9e7e2cc70dfa3400109d0ba439a0a9e717074a0 (diff) | |
download | llvm-d1946f7afe06924fdf7d3c9ad4b111aef8294ee1.tar.gz llvm-d1946f7afe06924fdf7d3c9ad4b111aef8294ee1.tar.bz2 llvm-d1946f7afe06924fdf7d3c9ad4b111aef8294ee1.tar.xz |
Ranges in the .debug_range section need to have begin and end labels,
assert that this is so.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@197780 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/CodeGen/AsmPrinter/DwarfDebug.cpp | 21 |
1 files changed, 8 insertions, 13 deletions
diff --git a/lib/CodeGen/AsmPrinter/DwarfDebug.cpp b/lib/CodeGen/AsmPrinter/DwarfDebug.cpp index 1c6d1a760d..a30e8bbd95 100644 --- a/lib/CodeGen/AsmPrinter/DwarfDebug.cpp +++ b/lib/CodeGen/AsmPrinter/DwarfDebug.cpp @@ -2938,14 +2938,12 @@ void DwarfDebug::emitDebugRanges() { RE = List.getRanges().end(); RI != RE; ++RI) { const RangeSpan &Range = *RI; - // We occasionally have ranges without begin/end labels. - // FIXME: Verify and fix. const MCSymbol *Begin = Range.getStart(); const MCSymbol *End = Range.getEnd(); - Begin ? Asm->OutStreamer.EmitSymbolValue(Begin, Size) - : Asm->OutStreamer.EmitIntValue(0, Size); - End ? Asm->OutStreamer.EmitSymbolValue(End, Size) - : Asm->OutStreamer.EmitIntValue(0, Size); + assert(Begin && "Range without a begin symbol?"); + assert(End && "Range without an end symbol?"); + Asm->OutStreamer.EmitSymbolValue(Begin, Size); + Asm->OutStreamer.EmitSymbolValue(End, Size); } // And terminate the list with two 0 values. @@ -2960,15 +2958,12 @@ void DwarfDebug::emitDebugRanges() { const SmallVectorImpl<RangeSpan> &Ranges = TheCU->getRanges(); for (uint32_t i = 0, e = Ranges.size(); i != e; ++i) { RangeSpan Range = Ranges[i]; - - // We occasionally have ranges without begin/end labels. - // FIXME: Verify and fix. const MCSymbol *Begin = Range.getStart(); const MCSymbol *End = Range.getEnd(); - Begin ? Asm->OutStreamer.EmitSymbolValue(Begin, Size) - : Asm->OutStreamer.EmitIntValue(0, Size); - End ? Asm->OutStreamer.EmitSymbolValue(End, Size) - : Asm->OutStreamer.EmitIntValue(0, Size); + assert(Begin && "Range without a begin symbol?"); + assert(End && "Range without an end symbol?"); + Asm->OutStreamer.EmitSymbolValue(Begin, Size); + Asm->OutStreamer.EmitSymbolValue(End, Size); } // And terminate the list with two 0 values. Asm->OutStreamer.EmitIntValue(0, Size); |