From 09c5aab7f88178c1af92de0b00417416da9db6c1 Mon Sep 17 00:00:00 2001
From: Nico Weber <thakis@chromium.org>
Date: Sat, 17 Dec 2022 09:25:42 -0500
Subject: [PATCH] Revert "Reland 2nd attempt:  [lld-macho] Fix bug in reading
 cpuSubType field."

This reverts commit b08acee423935c16a1503335d78b84ccbbae4ef8.
Still breaks tests, see https://reviews.llvm.org/D139572#4003191
---
 lld/MachO/InputFiles.cpp  | 10 ++--------
 lld/test/MachO/fat-arch.s | 28 +++-------------------------
 2 files changed, 5 insertions(+), 33 deletions(-)

diff --git a/lld/MachO/InputFiles.cpp b/lld/MachO/InputFiles.cpp
index 174ad95..54a02f2 100644
--- a/lld/MachO/InputFiles.cpp
+++ b/lld/MachO/InputFiles.cpp
@@ -231,14 +231,8 @@ std::optional<MemoryBufferRef> macho::readFile(StringRef path) {
       return std::nullopt;
     }
 
-    uint32_t cpuType = read32be(&arch[i].cputype);
-    uint32_t cpuSubtype =
-        read32be(&arch[i].cpusubtype) & ~MachO::CPU_SUBTYPE_MASK;
-
-    // FIXME: LD64 has a more complex fallback logic here.
-    // Consider implementing that as well?
-    if (cpuType != static_cast<uint32_t>(target->cpuType) ||
-        cpuSubtype != target->cpuSubtype)
+    if (read32be(&arch[i].cputype) != static_cast<uint32_t>(target->cpuType) ||
+        read32be(&arch[i].cpusubtype) != target->cpuSubtype)
       continue;
 
     uint32_t offset = read32be(&arch[i].offset);
diff --git a/lld/test/MachO/fat-arch.s b/lld/test/MachO/fat-arch.s
index 313b115..33928b1 100644
--- a/lld/test/MachO/fat-arch.s
+++ b/lld/test/MachO/fat-arch.s
@@ -1,38 +1,16 @@
-# REQUIRES: x86,aarch64
-## FIXME: The tests doesn't run on windows right now because of llvm-mc (can't produce triple=arm64-apple-macos11.0)
-# UNSUPPORTED: system-windows
-
+# REQUIRES: x86
 # RUN: llvm-mc -filetype=obj -triple=i386-apple-darwin %s -o %t.i386.o
 # RUN: llvm-mc -filetype=obj -triple=x86_64-apple-darwin %s -o %t.x86_64.o
-# RUN: llvm-mc -filetype=obj -triple=arm64-apple-macos11.0 %s -o %t.arm64.o
-
 # RUN: llvm-lipo %t.i386.o %t.x86_64.o -create -o %t.fat.o
 # RUN: %lld -o /dev/null %t.fat.o
+
 # RUN: llvm-lipo %t.i386.o -create -o %t.noarch.o
 # RUN: not %lld -o /dev/null %t.noarch.o 2>&1 | \
 # RUN:    FileCheck %s -DFILE=%t.noarch.o
 # CHECK: error: unable to find matching architecture in [[FILE]]
 
-## Validates that we read the cpu-subtype correctly from a fat exec.
-# RUN: %lld -o %t.x86_64.out %t.x86_64.o
-# RUN: %lld -arch arm64 -o %t.arm64.out %t.arm64.o
-# RUN: llvm-lipo %t.x86_64.out %t.arm64.out -create -o %t.fat.exec.out
-# RUN: %lld %t.x86_64.o -bundle_loader %t.fat.exec.out -bundle -o %t.fat.bundle
-
-# RUN: llvm-otool -h %t.fat.bundle -f %t.fat.exec.out | FileCheck %s --check-prefix=CPU-SUB
-# CPU-SUB: Fat headers
-# CPU-SUB: nfat_arch 2
-# CPU-SUB: architecture 0
-# CPU-SUB-NEXT:    cputype 16777223
-# CPU-SUB-NEXT:    cpusubtype 3
-# CPU-SUB: architecture 1
-# CPU-SUB-NEXT:    cputype 16777228
-# CPU-SUB-NEXT:    cpusubtype 0
-
-# CPU-SUB:            magic     cputype      cpusubtype   caps      filetype   ncmds sizeofcmds      flags
-# CPU-SUB-NEXT:  0xfeedfacf     16777223              3  0x{{.+}}    {{.+}}  {{.+}}    {{.+}}      {{.+}}
-
 .text
 .global _main
 _main:
+  mov $0, %eax
   ret
-- 
2.7.4