build: remove no longer needed arm64 macos patches (#26650)

patches/v8/.patches

@@ -7,7 +7,5 @@ workaround_an_undefined_symbol_error.patch
 do_not_export_private_v8_symbols_on_windows.patch
 revert_cleanup_switch_offset_of_to_offsetof_where_possible.patch
 fix_build_deprecated_attirbute_for_older_msvc_versions.patch
-chore_add_v8_apple_silicon_patches.patch
-fix_use_proper_page_size_for_mac_arm64.patch
 fix_correct_calling_convention_for_windows_on_arm.patch
 perf_make_getpositioninfoslow_faster.patch

patches/v8/chore_add_v8_apple_silicon_patches.patch

@@ -1,134 +0,0 @@
-From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
-From: Samuel Attard <[email protected]>
-Date: Mon, 6 Jul 2020 20:47:43 -0700
-Subject: chore: add v8 apple silicon patches
-
-Once these are available upstream we can remove this patch.
-
-diff --git a/src/codegen/arm64/macro-assembler-arm64.cc b/src/codegen/arm64/macro-assembler-arm64.cc
-index fef1758aaa81a85009461839ec9aa816e7d196a6..c0f7a1cf5fd4787896b06bce911aabf17b290e88 100644
---- a/src/codegen/arm64/macro-assembler-arm64.cc
-+++ b/src/codegen/arm64/macro-assembler-arm64.cc
-@@ -2975,6 +2975,35 @@ void TurboAssembler::PrintfNoPreserve(const char* format,
- 
-   int arg_count = kPrintfMaxArgCount;
- 
-+#if V8_OS_MACOSX && !USE_SIMULATOR
-+  CPURegList tmp_list = kCallerSaved;
-+  tmp_list.Remove(x0);      // Used to pass the format string.
-+  tmp_list.Remove(arg0, arg1, arg2, arg3);
-+
-+  // Override the MacroAssembler's scratch register list. The lists will be
-+  // reset automatically at the end of the UseScratchRegisterScope.
-+  UseScratchRegisterScope temps(this);
-+  TmpList()->set_list(tmp_list.list());
-+
-+  VRegister temp_D = temps.AcquireD();
-+
-+  // https://developer.apple.com/library/archive/documentation/Xcode/Conceptual/iPhoneOSABIReference/Articles/ARM64FunctionCallingConventions.html#//apple_ref/doc/uid/TP40013702-SW1
-+  Claim(kPrintfMaxArgCount, 8);
-+  int64_t offset = 0;
-+  for (unsigned i = 0; i < kPrintfMaxArgCount; i++) {
-+    CPURegister arg = args[i];
-+    if (arg.IsNone()) {
-+      break;
-+    }
-+    if (arg.IsS()) {
-+      fcvt(temp_D, arg.S());
-+      arg = temp_D;
-+    }
-+    // FIXME: Use stp.
-+    str(arg, MemOperand(sp, offset, Offset));
-+    offset += 8;
-+  }
-+#else
-   // The PCS varargs registers for printf. Note that x0 is used for the printf
-   // format string.
-   static const CPURegList kPCSVarargs =
-@@ -3085,7 +3114,7 @@ void TurboAssembler::PrintfNoPreserve(const char* format,
-     }
- #endif
-   }
--
-+#endif
-   // Load the format string into x0, as per the procedure-call standard.
-   //
-   // To make the code as portable as possible, the format string is encoded
-@@ -3107,6 +3136,10 @@ void TurboAssembler::PrintfNoPreserve(const char* format,
-   }
- 
-   CallPrintf(arg_count, pcs);
-+
-+#if V8_OS_MACOSX && !USE_SIMULATOR
-+  Drop(kPrintfMaxArgCount, 8);
-+#endif
- }
- 
- void TurboAssembler::CallPrintf(int arg_count, const CPURegister* args) {
-diff --git a/src/compiler/backend/arm64/instruction-selector-arm64.cc b/src/compiler/backend/arm64/instruction-selector-arm64.cc
-index fac7f9c1d1146b36b57aa70565fc86a1604887c8..6c13a0747009e38fd574ba94d38b6654fdd70edf 100644
---- a/src/compiler/backend/arm64/instruction-selector-arm64.cc
-+++ b/src/compiler/backend/arm64/instruction-selector-arm64.cc
-@@ -1898,6 +1898,7 @@ void InstructionSelector::EmitPrepareArguments(
- 
-   // Poke the arguments into the stack.
-   while (slot >= 0) {
-+    // FIXME: In the Apple ARM64 ABI parameters should be packed on the stack.
-     PushParameter input0 = (*arguments)[slot];
-     PushParameter input1 = slot > 0 ? (*arguments)[slot - 1] : PushParameter();
-     // Emit a poke-pair if consecutive parameters have the same type.
-diff --git a/src/flags/flag-definitions.h b/src/flags/flag-definitions.h
-index ab689283e970a79bc02dbeee7a4a74720265fcd7..12e54765d316f97297ad0a999ca32c5d07872580 100644
---- a/src/flags/flag-definitions.h
-+++ b/src/flags/flag-definitions.h
-@@ -763,7 +763,12 @@ DEFINE_INT(wasm_num_compilation_tasks, 128,
-            "maximum number of parallel compilation tasks for wasm")
- DEFINE_DEBUG_BOOL(trace_wasm_native_heap, false,
-                   "trace wasm native heap events")
--DEFINE_BOOL(wasm_write_protect_code_memory, false,
-+#if V8_OS_MACOSX && V8_TARGET_ARCH_ARM64
-+#define V8_DEFAULT_WASM_WRITE_PROTECT_CODE_MEMORY true
-+#else
-+#define V8_DEFAULT_WASM_WRITE_PROTECT_CODE_MEMORY false
-+#endif
-+DEFINE_BOOL(wasm_write_protect_code_memory, V8_DEFAULT_WASM_WRITE_PROTECT_CODE_MEMORY,
-             "write protect code memory on the wasm native heap")
- DEFINE_DEBUG_BOOL(trace_wasm_serialization, false,
-                   "trace serialization/deserialization")
-diff --git a/src/wasm/function-compiler.cc b/src/wasm/function-compiler.cc
-index 8b41a90992f58c535df285e7fa569b51ae31f85d..1ff813abf44d25e8f2fd84b236623b74913684fb 100644
---- a/src/wasm/function-compiler.cc
-+++ b/src/wasm/function-compiler.cc
-@@ -258,6 +258,7 @@ void WasmCompilationUnit::CompileWasmFunction(Isolate* isolate,
-       isolate->counters(), detected);
-   if (result.succeeded()) {
-     WasmCodeRefScope code_ref_scope;
-+    NativeModuleModificationScope native_module_modification_scope(native_module);
-     native_module->PublishCode(
-         native_module->AddCompiledCode(std::move(result)));
-   } else {
-diff --git a/src/wasm/wasm-code-manager.h b/src/wasm/wasm-code-manager.h
-index 5e8ed5475bb064cb657069242ce7517de7f5f8e2..5652bb407da3646e55c42492b3e9facec4556367 100644
---- a/src/wasm/wasm-code-manager.h
-+++ b/src/wasm/wasm-code-manager.h
-@@ -886,7 +886,7 @@ class V8_EXPORT_PRIVATE WasmCodeManager final {
- //    and even if we did, the resulting set of pages may be fragmented.
- //    Currently, we try and keep the number of syscalls low.
- // -  similar argument for debug time.
--class NativeModuleModificationScope final {
-+class V8_EXPORT_PRIVATE NativeModuleModificationScope final {
-  public:
-   explicit NativeModuleModificationScope(NativeModule* native_module);
-   ~NativeModuleModificationScope();
-diff --git a/src/wasm/wasm-objects.cc b/src/wasm/wasm-objects.cc
-index cf78ab5ff3286d6c07527a323eac4213255fe825..b89433d445f46110ba6cacfc6ec930d31171fd1f 100644
---- a/src/wasm/wasm-objects.cc
-+++ b/src/wasm/wasm-objects.cc
-@@ -1512,6 +1512,7 @@ void WasmInstanceObject::ImportWasmJSFunctionIntoTable(
-     wasm::WasmCompilationResult result = compiler::CompileWasmImportCallWrapper(
-         isolate->wasm_engine(), &env, kind, sig, false,
-         shared.internal_formal_parameter_count());
-+    wasm::NativeModuleModificationScope native_module_modification_scope(native_module);
-     std::unique_ptr<wasm::WasmCode> wasm_code = native_module->AddCode(
-         result.func_index, result.code_desc, result.frame_slot_count,
-         result.tagged_parameter_slots,

patches/v8/fix_use_proper_page_size_for_mac_arm64.patch

@@ -1,43 +0,0 @@
-From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
-From: John Kleinschmidt <[email protected]>
-Date: Wed, 8 Jul 2020 10:46:13 -0400
-Subject: fix: use proper page size for mac arm64
-
-This fix temporarily sets page size to 16k for mac arm64 so that mksnapshot can run on x64
-and generate a mac arm64 snapshot
-
-diff --git a/src/base/platform/platform-posix.cc b/src/base/platform/platform-posix.cc
-index 89173b593a6ce887c0f3cea082f2d31e17f8f5db..f233d83286dd82559ee3573b7adc525adf7ce377 100644
---- a/src/base/platform/platform-posix.cc
-+++ b/src/base/platform/platform-posix.cc
-@@ -167,9 +167,19 @@ void* Allocate(void* hint, size_t size, OS::MemoryPermission access,
-                PageType page_type) {
-   int prot = GetProtectionFromMemoryPermission(access);
-   int flags = GetFlagsForMemoryPermission(access, page_type);
-+#if defined(__APPLE__) && V8_TARGET_ARCH_ARM64 && defined(__x86_64__)
-+  // XXX: This logic is simple and leaky as it is only used for mksnapshot.
-+  size_t alignment = 16384;
-+  void* result = mmap(hint, size + alignment, prot, flags, kMmapFd,
-+                      kMmapFdOffset);
-+  if (result == MAP_FAILED) return nullptr;
-+  return reinterpret_cast<void*>(
-+      RoundUp(reinterpret_cast<uintptr_t>(result), alignment));
-+#else
-   void* result = mmap(hint, size, prot, flags, kMmapFd, kMmapFdOffset);
-   if (result == MAP_FAILED) return nullptr;
-   return result;
-+#endif
- }
- 
- #endif  // !V8_OS_FUCHSIA
-@@ -226,7 +236,9 @@ void OS::Initialize(bool hard_abort, const char* const gc_fake_mmap) {
- }
- 
- int OS::ActivationFrameAlignment() {
--#if V8_TARGET_ARCH_ARM
-+#if defined(__APPLE__) && V8_TARGET_ARCH_ARM
-+  return 4;
-+#elif V8_TARGET_ARCH_ARM
-   // On EABI ARM targets this is required for fp correctness in the
-   // runtime system.
-   return 8;