[llvm] [SDAG] Ensure load is included in output chain of sincos expansion (PR #140525)

Benjamin Maxwell via llvm-commits llvm-commits at lists.llvm.org
Mon May 19 03:54:44 PDT 2025


================
@@ -294,10 +294,10 @@ def scrub_asm_x86(asm, args):
     else:
         asm = SCRUB_X86_SHUFFLES_RE.sub(r"\1 {{.*#+}} \2", asm)
 
-    # Detect stack spills and reloads and hide their exact offset and whether
-    # they used the stack pointer or frame pointer.
-    asm = SCRUB_X86_SPILL_RELOAD_RE.sub(r"{{[-0-9]+}}(%\1{{[sb]}}p)\2", asm)
     if getattr(args, "x86_scrub_sp", True):
+        # Detect stack spills and reloads and hide their exact offset and whether
+        # they used the stack pointer or frame pointer.
+        asm = SCRUB_X86_SPILL_RELOAD_RE.sub(r"{{[-0-9]+}}(%\1{{[sb]}}p)\2", asm)
----------------
MacDue wrote:

Removed (along with the claim this test is auto-generated). 

https://github.com/llvm/llvm-project/pull/140525


More information about the llvm-commits mailing list