[llvm-mc] Don't print initial .text for disassembler

``` % echo 90 | llvm-mc -triple=x86_64 --disassemble --hex .text nop ``` The initial `.text` kludge is due `initSection`, which is actually only needed by AIX XCOFF for its `getCurrentSectionOnly()` use in MCAsmStreamer::emitInstruction (https://reviews.llvm.org/D95518). Adjust MCAsmStreamer::emitInstruction to not trigger failures on ``` echo 7c4303a6 | llvm-mc --cdis --hex --triple=powerpc-aix-ibm-xcoff ``` Pull Request: llvm/llvm-project#120185
intel · Dec 17, 2024 · 495bd4c · 495bd4c
1 parent 6db1b20
commit 495bd4c
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 11 deletions.
diff --git a/llvm/lib/MC/MCAsmStreamer.cpp b/llvm/lib/MC/MCAsmStreamer.cpp
@@ -2428,10 +2428,7 @@ void MCAsmStreamer::AddEncodingComment(const MCInst &Inst,
 
 void MCAsmStreamer::emitInstruction(const MCInst &Inst,
                                     const MCSubtargetInfo &STI) {
-  assert(getCurrentSectionOnly() &&
-         "Cannot emit contents before setting section!");
-
-  if (!MAI->usesDwarfFileAndLocDirectives())
+  if (!MAI->usesDwarfFileAndLocDirectives() && CurFrag)
     // Now that a machine instruction has been assembled into this section, make
     // a line entry for any .loc directive that has been seen.
     MCDwarfLineEntry::make(this, getCurrentSectionOnly());

diff --git a/llvm/test/MC/Disassembler/AArch64/colored.txt b/llvm/test/MC/Disassembler/AArch64/colored.txt
@@ -1,6 +1,7 @@
 # UNSUPPORTED: system-windows
 # RUN: llvm-mc -triple=aarch64 -mattr=+all --cdis %s | FileCheck %s --strict-whitespace --match-full-lines
 
+# CHECK-NOT:{{.}}
 0xa1 0x00 0x00 0x54
 # CHECK:	b.ne	[0;31m#20[0m
 0x00 0x7c 0x00 0x13

diff --git a/llvm/test/MC/Disassembler/ARM/arm-trustzone.txt b/llvm/test/MC/Disassembler/ARM/arm-trustzone.txt
@@ -1,4 +1,4 @@
-# RUN: llvm-mc -triple=armv7-apple-darwin -mcpu=cortex-a8 -disassemble -mattr=-trustzone < %s | FileCheck %s -check-prefix=NOTZ
+# RUN: llvm-mc -triple=armv7-apple-darwin -mcpu=cortex-a8 -disassemble -mattr=-trustzone %s 2>&1 | FileCheck %s -check-prefix=NOTZ
 # RUN: llvm-mc -triple=armv7-apple-darwin -mcpu=cortex-a8 -disassemble -mattr=trustzone < %s | FileCheck %s -check-prefix=TZ
 
 
@@ -9,8 +9,7 @@
 0x7f 0x00 0x60 0xe1
 0x70 0x00 0x60 0x01
 
-# NOTZ-NOT: smc #15
-# NOTZ-NOT: smceq #0
+# NOTZ-COUNT-2: warning: invalid instruction encoding
 # TZ: smc #15
 # TZ: smceq #0
 
diff --git a/llvm/test/tools/llvm-mc/line_end_with_space.test b/llvm/test/tools/llvm-mc/line_end_with_space.test
@@ -1,3 +1,3 @@
 # Test that llvm-mc doesn't crash when disassembling a line that ends with a
 # space.
-RUN: echo -n ' ' | llvm-mc -disassemble
+RUN: echo -n ' ' | llvm-mc -disassemble | count 0
diff --git a/llvm/tools/llvm-mc/Disassembler.cpp b/llvm/tools/llvm-mc/Disassembler.cpp
@@ -168,9 +168,6 @@ int Disassembler::disassemble(const Target &T, const std::string &Triple,
     return -1;
   }
 
-  // Set up initial section manually here
-  Streamer.initSections(false, STI);
-
   bool ErrorOccurred = false;
 
   // Convert the input to a vector for disassembly.