[Lldb-commits] [lldb] Partially revert "[NFCI][lldb][test][asm] Enable AT&T syntax explicitly (#166770)" (PR #172233)

Raul Tambre via lldb-commits lldb-commits at lists.llvm.org
Sun Dec 14 14:26:43 PST 2025


https://github.com/tambry created https://github.com/llvm/llvm-project/pull/172233

Flag changes reverted as those require the X86 target to be enabled.  
Don't have time to test fixes as I need to go to sleep so will revert for now.

Reverts: 423919d31f4b55f22b09cd5066534f7c91e71d4b

>From b3977013a4d6147b4d04a7a0da27cc55c436aa72 Mon Sep 17 00:00:00 2001
From: Raul Tambre <raul at tambre.ee>
Date: Mon, 15 Dec 2025 00:22:44 +0200
Subject: [PATCH] Partially revert "[NFCI][lldb][test][asm] Enable AT&T syntax
 explicitly (#166770)"

Flag changes reverted as those require the X86 target to be enabled.
Don't have time to test fixes as I need to go to sleep so will revert for now.

Reverts: 423919d31f4b55f22b09cd5066534f7c91e71d4b
---
 lldb/packages/Python/lldbsuite/test/make/Makefile.rules | 7 -------
 lldb/test/Shell/helper/toolchain.py                     | 4 ----
 2 files changed, 11 deletions(-)

diff --git a/lldb/packages/Python/lldbsuite/test/make/Makefile.rules b/lldb/packages/Python/lldbsuite/test/make/Makefile.rules
index 1b967219134d8..55dbd3934860f 100644
--- a/lldb/packages/Python/lldbsuite/test/make/Makefile.rules
+++ b/lldb/packages/Python/lldbsuite/test/make/Makefile.rules
@@ -320,13 +320,6 @@ ifeq "$(MAKE_GMODULES)" "YES"
 	CXXFLAGS += $(MANDATORY_MODULE_BUILD_CFLAGS)
 endif
 
-# Our files use x86 AT&T assembly throughout.
-# Enable it explicitly so any local Clang preference for Intel syntax gets overriden.
-ifeq ($(CC_TYPE), clang)
-	CFLAGS += -mllvm -x86-asm-syntax=att
-	CXXFLAGS += -mllvm -x86-asm-syntax=att
-endif
-
 CFLAGS += $(CFLAGS_EXTRAS)
 CXXFLAGS += -std=c++11 $(CFLAGS) $(ARCH_CXXFLAGS)
 # Copy common options to the linker flags (dwarf, arch. & etc).
diff --git a/lldb/test/Shell/helper/toolchain.py b/lldb/test/Shell/helper/toolchain.py
index dd8ca48b7d76e..0c8c39d37e089 100644
--- a/lldb/test/Shell/helper/toolchain.py
+++ b/lldb/test/Shell/helper/toolchain.py
@@ -242,10 +242,6 @@ def use_support_substitutions(config):
     # The clang module cache is used for building inferiors.
     host_flags += ["-fmodules-cache-path={}".format(config.clang_module_cache)]
 
-    # Our files use x86 AT&T assembly throughout.
-    # Enable it explicitly so any local Clang preference for Intel syntax gets overriden.
-    host_flags += ["-mllvm", "-x86-asm-syntax=att"]
-
     if config.cmake_sysroot:
         host_flags += ["--sysroot={}".format(config.cmake_sysroot)]
 



More information about the lldb-commits mailing list