Review feedback (improved comments, mandatory REQUIRES)

Author: jmorse

llvm/lib/MC/MCObjectStreamer.cpp

@@ -465,10 +465,9 @@ void MCObjectStreamer::emitDwarfAdvanceLineAddr(int64_t LineDelta,
     return;
   }
 
-  // If the two labels are within the same fragment and it's a plain data
-  // fragment, then the address-offset is already a fixed constant and is not
-  // relaxable. Emit the advance-line-addr data immediately to save time and
-  // memory.
+  // If the two labels are within the same fragment, then the address-offset is
+  // already a fixed constant and is not relaxable. Emit the advance-line-addr
+  // data immediately to save time and memory.
   if (auto OptAddrDelta = absoluteSymbolDiff(Label, LastLabel)) {
     SmallString<16> Tmp;
     MCDwarfLineAddr::encode(getContext(), Assembler->getDWARFLinetableParams(),

llvm/test/DebugInfo/X86/debug-line-in-one-fragment.ll

@@ -1,6 +1,8 @@
 ; RUN: llc %s -o %t.o -filetype=obj
 ; RUN: llvm-dwarfdump --debug-line %t.o | FileCheck %s --check-prefix=LINES
 ; RUN: llc %s -o %t.o -filetype=obj -debug-only=mc-dump 2>&1 | FileCheck %s --check-prefix=FRAGMENTS
+;
+; REQUIRES: asserts
 
 ;; Test (using mc-dump debug output) that .debug_line can be arranged in memory
 ;; using a single data fragment for a simple function, instead of using multiple