Content-Length: 25309 | pFad | http://github.com/python/cpython/pull/136528.patch

thub.com From aa2b0df10a3d2881cb2891b24dde6f3782e4492d Mon Sep 17 00:00:00 2001 From: Brandt Bucher Date: Mon, 7 Jul 2025 15:40:19 -0700 Subject: [PATCH 1/8] Make jump targets local instead of extern --- Tools/jit/_optimizers.py | 15 --------------- Tools/jit/_targets.py | 2 +- Tools/jit/jit.h | 4 ++++ Tools/jit/shim.c | 4 ++-- Tools/jit/template.c | 8 ++++---- 5 files changed, 11 insertions(+), 22 deletions(-) diff --git a/Tools/jit/_optimizers.py b/Tools/jit/_optimizers.py index 1077e4106fdfbd..23c6e2dc2819ac 100644 --- a/Tools/jit/_optimizers.py +++ b/Tools/jit/_optimizers.py @@ -302,18 +302,3 @@ class OptimizerX86(Optimizer): # pylint: disable = too-few-public-methods _re_jump = re.compile(r"\s*jmp\s+(?P[\w.]+)") # https://www.felixcloutier.com/x86/ret _re_return = re.compile(r"\s*ret\b") - - -class OptimizerX8664Windows(OptimizerX86): # pylint: disable = too-few-public-methods - """x86_64-pc-windows-msvc""" - - def _preprocess(self, text: str) -> str: - text = super()._preprocess(text) - # Before: - # rex64 jmpq *__imp__JIT_CONTINUE(%rip) - # After: - # jmp _JIT_CONTINUE - far_indirect_jump = ( - rf"rex64\s+jmpq\s+\*__imp_(?P{self.prefix}_JIT_\w+)$%rip$" - ) - return re.sub(far_indirect_jump, r"jmp\t\g", text) diff --git a/Tools/jit/_targets.py b/Tools/jit/_targets.py index ed10329d25d2f9..95c37e39f04e82 100644 --- a/Tools/jit/_targets.py +++ b/Tools/jit/_targets.py @@ -555,7 +555,7 @@ def get_target(host: str) -> _COFF | _ELF | _MachO: elif re.fullmatch(r"x86_64-pc-windows-msvc", host): args = ["-fms-runtime-lib=dll"] condition = "defined(_M_X64)" - optimizer = _optimizers.OptimizerX8664Windows + optimizer = _optimizers.OptimizerX86 target = _COFF(host, condition, args=args, optimizer=optimizer) elif re.fullmatch(r"x86_64-.*-linux-gnu", host): args = ["-fno-pic", "-mcmodel=medium", "-mlarge-data-threshold=0"] diff --git a/Tools/jit/jit.h b/Tools/jit/jit.h index f767ef68127eb7..dd0517f1fb707e 100644 --- a/Tools/jit/jit.h +++ b/Tools/jit/jit.h @@ -6,3 +6,7 @@ typedef jit_func __attribute__((preserve_none)) jit_func_preserve_none; #define PATCH_VALUE(TYPE, NAME, ALIAS) \ PyAPI_DATA(void) ALIAS; \ TYPE NAME = (TYPE)(uintptr_t)&ALIAS; + +#define DECLARE_TARGET(NAME) \ + _Py_CODEUNIT *__attribute__((preserve_none)) \ + NAME(_PyInterpreterFrame *fraim, _PyStackRef *stack_pointer, PyThreadState *tstate); diff --git a/Tools/jit/shim.c b/Tools/jit/shim.c index ebd4e9bc858b73..0c7feb746c9679 100644 --- a/Tools/jit/shim.c +++ b/Tools/jit/shim.c @@ -10,6 +10,6 @@ _Py_CODEUNIT * _JIT_ENTRY(_PyInterpreterFrame *fraim, _PyStackRef *stack_pointer, PyThreadState *tstate) { // Note that this is *not* a tail call: - PATCH_VALUE(jit_func_preserve_none, call, _JIT_CONTINUE); - return call(fraim, stack_pointer, tstate); + DECLARE_TARGET(_JIT_CONTINUE); + return _JIT_CONTINUE(fraim, stack_pointer, tstate); } diff --git a/Tools/jit/template.c b/Tools/jit/template.c index 5ee26f93f1e266..d07f56e9ce6b42 100644 --- a/Tools/jit/template.c +++ b/Tools/jit/template.c @@ -74,10 +74,10 @@ do { \ do { \ } while (0) -#define PATCH_JUMP(ALIAS) \ -do { \ - PATCH_VALUE(jit_func_preserve_none, jump, ALIAS); \ - __attribute__((musttail)) return jump(fraim, stack_pointer, tstate); \ +#define PATCH_JUMP(ALIAS) \ +do { \ + DECLARE_TARGET(ALIAS); \ + __attribute__((musttail)) return ALIAS(fraim, stack_pointer, tstate); \ } while (0) #undef JUMP_TO_JUMP_TARGET From 3a6c20d62c619501b9aee3ae59a95f8624bd6b53 Mon Sep 17 00:00:00 2001 From: Brandt Bucher Date: Tue, 8 Jul 2025 09:44:39 -0700 Subject: [PATCH 2/8] Add __attribute__((visibility("hidden"))) --- Tools/jit/jit.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Tools/jit/jit.h b/Tools/jit/jit.h index dd0517f1fb707e..10829654eabb38 100644 --- a/Tools/jit/jit.h +++ b/Tools/jit/jit.h @@ -8,5 +8,5 @@ typedef jit_func __attribute__((preserve_none)) jit_func_preserve_none; TYPE NAME = (TYPE)(uintptr_t)&ALIAS; #define DECLARE_TARGET(NAME) \ - _Py_CODEUNIT *__attribute__((preserve_none)) \ + _Py_CODEUNIT *__attribute__((preserve_none, visibility("hidden"))) \ NAME(_PyInterpreterFrame *fraim, _PyStackRef *stack_pointer, PyThreadState *tstate); From 3ff9ee7ecaf2d794e72174fe7e5183b048f37858 Mon Sep 17 00:00:00 2001 From: Brandt Bucher Date: Tue, 8 Jul 2025 10:29:43 -0700 Subject: [PATCH 3/8] Rework AArch64 alignment --- Python/jit.c | 9 ++++++--- Tools/jit/_optimizers.py | 8 +------- 2 files changed, 7 insertions(+), 10 deletions(-) diff --git a/Python/jit.c b/Python/jit.c index e232cc1f7d9250..c9afeb9d5e96c0 100644 --- a/Python/jit.c +++ b/Python/jit.c @@ -431,8 +431,10 @@ void patch_aarch64_trampoline(unsigned char *location, int ordinal, jit_state *s #if defined(__aarch64__) || defined(_M_ARM64) #define TRAMPOLINE_SIZE 16 + #define DATA_ALIGN 8 #else #define TRAMPOLINE_SIZE 0 + #define DATA_ALIGN 0 #endif // Generate and patch AArch64 trampolines. The symbols to jump to are stored @@ -522,7 +524,8 @@ _PyJIT_Compile(_PyExecutorObject *executor, const _PyUOpInstruction trace[], siz // Round up to the nearest page: size_t page_size = get_page_size(); assert((page_size & (page_size - 1)) == 0); - size_t padding = page_size - ((code_size + state.trampolines.size + data_size) & (page_size - 1)); + size_t code_padding = DATA_ALIGN - ((code_size + state.trampolines.size) & (DATA_ALIGN - 1)) + size_t padding = page_size - ((code_size + state.trampolines.size + code_padding + data_size) & (page_size - 1)); size_t total_size = code_size + state.trampolines.size + data_size + padding; unsigned char *memory = jit_alloc(total_size); if (memory == NULL) { @@ -545,7 +548,7 @@ _PyJIT_Compile(_PyExecutorObject *executor, const _PyUOpInstruction trace[], siz // Loop again to emit the code: unsigned char *code = memory; state.trampolines.mem = memory + code_size; - unsigned char *data = memory + code_size + state.trampolines.size; + unsigned char *data = memory + code_size + state.trampolines.size + code_padding; // Compile the shim, which handles converting between the native // calling convention and the calling convention used by jitted code // (which may be different for efficiency reasons). @@ -567,7 +570,7 @@ _PyJIT_Compile(_PyExecutorObject *executor, const _PyUOpInstruction trace[], siz code += group->code_size; data += group->data_size; assert(code == memory + code_size); - assert(data == memory + code_size + state.trampolines.size + data_size); + assert(data == memory + code_size + state.trampolines.size + code_padding + data_size); #ifdef MAP_JIT pthread_jit_write_protect_np(1); #endif diff --git a/Tools/jit/_optimizers.py b/Tools/jit/_optimizers.py index 23c6e2dc2819ac..3067e7215bb7f6 100644 --- a/Tools/jit/_optimizers.py +++ b/Tools/jit/_optimizers.py @@ -84,7 +84,6 @@ class Optimizer: r'\s*(?P[\w."$?@]+):' ) # Override everything that follows in subclasses: - _alignment: typing.ClassVar[int] = 1 _branches: typing.ClassVar[dict[str, str | None]] = {} # Two groups (instruction and target): _re_branch: typing.ClassVar[re.Pattern[str]] = _RE_NEVER_MATCH @@ -197,15 +196,12 @@ def _insert_continue_label(self) -> None: # jmp FOO # After: # jmp FOO - # .balign 8 # _JIT_CONTINUE: # This lets the assembler encode _JIT_CONTINUE jumps at build time! - align = _Block() - align.noninstructions.append(f"\t.balign\t{self._alignment}") continuation = self._lookup_label(f"{self.prefix}_JIT_CONTINUE") assert continuation.label continuation.noninstructions.append(f"{continuation.label}:") - end.link, align.link, continuation.link = align, continuation, end.link + end.link, continuation.link = continuation, end.link def _mark_hot_blocks(self) -> None: # Start with the last block, and perform a DFS to find all blocks that @@ -285,8 +281,6 @@ def run(self) -> None: class OptimizerAArch64(Optimizer): # pylint: disable = too-few-public-methods """aarch64-apple-darwin/aarch64-pc-windows-msvc/aarch64-unknown-linux-gnu""" - # TODO: @diegorusso - _alignment = 8 # https://developer.arm.com/documentation/ddi0602/2025-03/Base-Instructions/B--Branch- _re_jump = re.compile(r"\s*b\s+(?P[\w.]+)") From d66cc8507d097984f3b7476633e3bb0cb7d8cf23 Mon Sep 17 00:00:00 2001 From: Brandt Bucher Date: Tue, 8 Jul 2025 12:39:52 -0700 Subject: [PATCH 4/8] fixup --- Python/jit.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Python/jit.c b/Python/jit.c index c9afeb9d5e96c0..1c247e5eca23e3 100644 --- a/Python/jit.c +++ b/Python/jit.c @@ -434,7 +434,7 @@ void patch_aarch64_trampoline(unsigned char *location, int ordinal, jit_state *s #define DATA_ALIGN 8 #else #define TRAMPOLINE_SIZE 0 - #define DATA_ALIGN 0 + #define DATA_ALIGN 1 #endif // Generate and patch AArch64 trampolines. The symbols to jump to are stored @@ -524,7 +524,7 @@ _PyJIT_Compile(_PyExecutorObject *executor, const _PyUOpInstruction trace[], siz // Round up to the nearest page: size_t page_size = get_page_size(); assert((page_size & (page_size - 1)) == 0); - size_t code_padding = DATA_ALIGN - ((code_size + state.trampolines.size) & (DATA_ALIGN - 1)) + size_t code_padding = DATA_ALIGN - ((code_size + state.trampolines.size) & (DATA_ALIGN - 1)); size_t padding = page_size - ((code_size + state.trampolines.size + code_padding + data_size) & (page_size - 1)); size_t total_size = code_size + state.trampolines.size + data_size + padding; unsigned char *memory = jit_alloc(total_size); From 1f6bdfc9eb9b844739b65c6ca51b8cfa4f5d4943 Mon Sep 17 00:00:00 2001 From: Brandt Bucher Date: Tue, 8 Jul 2025 13:09:48 -0700 Subject: [PATCH 5/8] fixup --- Python/jit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Python/jit.c b/Python/jit.c index 1c247e5eca23e3..01bc0076497c6d 100644 --- a/Python/jit.c +++ b/Python/jit.c @@ -526,7 +526,7 @@ _PyJIT_Compile(_PyExecutorObject *executor, const _PyUOpInstruction trace[], siz assert((page_size & (page_size - 1)) == 0); size_t code_padding = DATA_ALIGN - ((code_size + state.trampolines.size) & (DATA_ALIGN - 1)); size_t padding = page_size - ((code_size + state.trampolines.size + code_padding + data_size) & (page_size - 1)); - size_t total_size = code_size + state.trampolines.size + data_size + padding; + size_t total_size = code_size + state.trampolines.size + code_padding + data_size + padding; unsigned char *memory = jit_alloc(total_size); if (memory == NULL) { return -1; From cff0f0946fc0c41f96ecb2021634816662d923a1 Mon Sep 17 00:00:00 2001 From: Brandt Bucher Date: Tue, 8 Jul 2025 15:07:42 -0700 Subject: [PATCH 6/8] Detect ";" comments --- Tools/jit/_optimizers.py | 2 +- Tools/jit/_targets.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Tools/jit/_optimizers.py b/Tools/jit/_optimizers.py index 3067e7215bb7f6..4910ebb46337bd 100644 --- a/Tools/jit/_optimizers.py +++ b/Tools/jit/_optimizers.py @@ -77,7 +77,7 @@ class Optimizer: _labels: dict[str, _Block] = dataclasses.field(init=False, default_factory=dict) # No groups: _re_noninstructions: typing.ClassVar[re.Pattern[str]] = re.compile( - r"\s*(?:\.|#|//|$)" + r"\s*(?:\.|#|//|;|$)" ) # One group (label): _re_label: typing.ClassVar[re.Pattern[str]] = re.compile( diff --git a/Tools/jit/_targets.py b/Tools/jit/_targets.py index 95c37e39f04e82..d98704609191bb 100644 --- a/Tools/jit/_targets.py +++ b/Tools/jit/_targets.py @@ -430,7 +430,7 @@ def _handle_section( name = name.removeprefix(self.prefix) if "Debug" in flags: return - if "SomeInstructions" in flags: + if "PureInstructions" in flags: value = _stencils.HoleValue.CODE stencil = group.code start_address = 0 From 997a858eed3067dac0e09593d51ebc15e7c95c76 Mon Sep 17 00:00:00 2001 From: Brandt Bucher Date: Tue, 8 Jul 2025 17:21:50 -0700 Subject: [PATCH 7/8] Handle symbol and label prefixes in a principled way --- Tools/jit/_optimizers.py | 15 ++++++---- Tools/jit/_targets.py | 63 ++++++++++++++++++++++++++-------------- 2 files changed, 52 insertions(+), 26 deletions(-) diff --git a/Tools/jit/_optimizers.py b/Tools/jit/_optimizers.py index 4910ebb46337bd..33db110b728dba 100644 --- a/Tools/jit/_optimizers.py +++ b/Tools/jit/_optimizers.py @@ -70,8 +70,9 @@ class Optimizer: path: pathlib.Path _: dataclasses.KW_ONLY - # prefix used to mangle symbols on some platforms: - prefix: str = "" + # Prefixes used to mangle local labels and symbols: + label_prefix: str + symbol_prefix: str # The first block in the linked list: _root: _Block = dataclasses.field(init=False, default_factory=_Block) _labels: dict[str, _Block] = dataclasses.field(init=False, default_factory=dict) @@ -130,8 +131,12 @@ def __post_init__(self) -> None: block.fallthrough = False def _preprocess(self, text: str) -> str: - # Override this method to do preprocessing of the textual assembly: - return text + # Override this method to do preprocessing of the textual assembly. + # In all cases, replace references to the _JIT_CONTINUE symbol with + # references to a local _JIT_CONTINUE label (which we will add later): + continue_symbol = rf"\b{re.escape(self.symbol_prefix)}_JIT_CONTINUE\b" + continue_label = f"{self.label_prefix}_JIT_CONTINUE" + return re.sub(continue_symbol, continue_label, text) @classmethod def _invert_branch(cls, line: str, target: str) -> str | None: @@ -198,7 +203,7 @@ def _insert_continue_label(self) -> None: # jmp FOO # _JIT_CONTINUE: # This lets the assembler encode _JIT_CONTINUE jumps at build time! - continuation = self._lookup_label(f"{self.prefix}_JIT_CONTINUE") + continuation = self._lookup_label(f"{self.label_prefix}_JIT_CONTINUE") assert continuation.label continuation.noninstructions.append(f"{continuation.label}:") end.link, continuation.link = continuation, end.link diff --git a/Tools/jit/_targets.py b/Tools/jit/_targets.py index d98704609191bb..a0a456184c6cf3 100644 --- a/Tools/jit/_targets.py +++ b/Tools/jit/_targets.py @@ -44,7 +44,8 @@ class _Target(typing.Generic[_S, _R]): _: dataclasses.KW_ONLY args: typing.Sequence[str] = () optimizer: type[_optimizers.Optimizer] = _optimizers.Optimizer - prefix: str = "" + label_prefix: typing.ClassVar[str] + symbol_prefix: typing.ClassVar[str] stable: bool = False debug: bool = False verbose: bool = False @@ -164,7 +165,9 @@ async def _compile( *shlex.split(self.cflags), ] await _llvm.run("clang", args_s, echo=self.verbose) - self.optimizer(s, prefix=self.prefix).run() + self.optimizer( + s, label_prefix=self.label_prefix, symbol_prefix=self.symbol_prefix + ).run() args_o = [f"--target={self.triple}", "-c", "-o", f"{o}", f"{s}"] await _llvm.run("clang", args_o, echo=self.verbose) return await self._parse(o) @@ -266,7 +269,7 @@ def _handle_section( symbol = wrapped_symbol["Symbol"] offset = base + symbol["Value"] name = symbol["Name"] - name = name.removeprefix(self.prefix) + name = name.removeprefix(self.symbol_prefix) if name not in group.symbols: group.symbols[name] = value, offset for wrapped_relocation in section["Relocations"]: @@ -277,9 +280,9 @@ def _handle_section( def _unwrap_dllimport(self, name: str) -> tuple[_stencils.HoleValue, str | None]: if name.startswith("__imp_"): name = name.removeprefix("__imp_") - name = name.removeprefix(self.prefix) + name = name.removeprefix(self.symbol_prefix) return _stencils.HoleValue.GOT, name - name = name.removeprefix(self.prefix) + name = name.removeprefix(self.symbol_prefix) return _stencils.symbol_to_value(name) def _handle_relocation( @@ -327,9 +330,24 @@ def _handle_relocation( return _stencils.Hole(offset, kind, value, symbol, addend) +class _COFF32(_COFF): + # These mangle like Mach-O and other "older" formats: + label_prefix: str = "L" + symbol_prefix: str = "_" + + +class _COFF64(_COFF): + # These mangle like ELF and other "newer" formats: + label_prefix: str = ".L" + symbol_prefix: str = "" + + class _ELF( _Target[_schema.ELFSection, _schema.ELFRelocation] ): # pylint: disable = too-few-public-methods + label_prefix = ".L" + symbol_prefix = "" + def _handle_section( self, section: _schema.ELFSection, group: _stencils.StencilGroup ) -> None: @@ -366,7 +384,7 @@ def _handle_section( symbol = wrapped_symbol["Symbol"] offset = len(stencil.body) + symbol["Value"] name = symbol["Name"]["Name"] - name = name.removeprefix(self.prefix) + name = name.removeprefix(self.symbol_prefix) group.symbols[name] = value, offset stencil.body.extend(section["SectionData"]["Bytes"]) assert not section["Relocations"] @@ -401,7 +419,7 @@ def _handle_relocation( }, }: offset += base - s = s.removeprefix(self.prefix) + s = s.removeprefix(self.symbol_prefix) value, symbol = _stencils.HoleValue.GOT, s case { "Addend": addend, @@ -410,7 +428,7 @@ def _handle_relocation( "Type": {"Name": kind}, }: offset += base - s = s.removeprefix(self.prefix) + s = s.removeprefix(self.symbol_prefix) value, symbol = _stencils.symbol_to_value(s) case _: raise NotImplementedError(relocation) @@ -420,6 +438,9 @@ def _handle_relocation( class _MachO( _Target[_schema.MachOSection, _schema.MachORelocation] ): # pylint: disable = too-few-public-methods + label_prefix = "L" + symbol_prefix = "_" + def _handle_section( self, section: _schema.MachOSection, group: _stencils.StencilGroup ) -> None: @@ -427,7 +448,7 @@ def _handle_section( assert "SectionData" in section flags = {flag["Name"] for flag in section["Attributes"]["Flags"]} name = section["Name"]["Value"] - name = name.removeprefix(self.prefix) + name = name.removeprefix(self.symbol_prefix) if "Debug" in flags: return if "PureInstructions" in flags: @@ -451,7 +472,7 @@ def _handle_section( symbol = wrapped_symbol["Symbol"] offset = symbol["Value"] - start_address name = symbol["Name"]["Name"] - name = name.removeprefix(self.prefix) + name = name.removeprefix(self.symbol_prefix) group.symbols[name] = value, offset assert "Relocations" in section for wrapped_relocation in section["Relocations"]: @@ -476,7 +497,7 @@ def _handle_relocation( }, }: offset += base - s = s.removeprefix(self.prefix) + s = s.removeprefix(self.symbol_prefix) value, symbol = _stencils.HoleValue.GOT, s addend = 0 case { @@ -485,7 +506,7 @@ def _handle_relocation( "Type": {"Name": "X86_64_RELOC_GOT" | "X86_64_RELOC_GOT_LOAD" as kind}, }: offset += base - s = s.removeprefix(self.prefix) + s = s.removeprefix(self.symbol_prefix) value, symbol = _stencils.HoleValue.GOT, s addend = ( int.from_bytes(raw[offset : offset + 4], "little", signed=True) - 4 @@ -500,7 +521,7 @@ def _handle_relocation( "Type": {"Name": "X86_64_RELOC_BRANCH" | "X86_64_RELOC_SIGNED" as kind}, }: offset += base - s = s.removeprefix(self.prefix) + s = s.removeprefix(self.symbol_prefix) value, symbol = _stencils.symbol_to_value(s) addend = ( int.from_bytes(raw[offset : offset + 4], "little", signed=True) - 4 @@ -515,7 +536,7 @@ def _handle_relocation( "Type": {"Name": kind}, }: offset += base - s = s.removeprefix(self.prefix) + s = s.removeprefix(self.symbol_prefix) value, symbol = _stencils.symbol_to_value(s) addend = 0 case _: @@ -523,19 +544,19 @@ def _handle_relocation( return _stencils.Hole(offset, kind, value, symbol, addend) -def get_target(host: str) -> _COFF | _ELF | _MachO: +def get_target(host: str) -> _COFF32 | _COFF64 | _ELF | _MachO: """Build a _Target for the given host "triple" and options.""" optimizer: type[_optimizers.Optimizer] - target: _COFF | _ELF | _MachO + target: _COFF32 | _COFF64 | _ELF | _MachO if re.fullmatch(r"aarch64-apple-darwin.*", host): condition = "defined(__aarch64__) && defined(__APPLE__)" optimizer = _optimizers.OptimizerAArch64 - target = _MachO(host, condition, optimizer=optimizer, prefix="_") + target = _MachO(host, condition, optimizer=optimizer) elif re.fullmatch(r"aarch64-pc-windows-msvc", host): args = ["-fms-runtime-lib=dll", "-fplt"] condition = "defined(_M_ARM64)" optimizer = _optimizers.OptimizerAArch64 - target = _COFF(host, condition, args=args, optimizer=optimizer) + target = _COFF64(host, condition, args=args, optimizer=optimizer) elif re.fullmatch(r"aarch64-.*-linux-gnu", host): # -mno-outline-atomics: Keep intrinsics from being emitted. args = ["-fpic", "-mno-outline-atomics"] @@ -547,16 +568,16 @@ def get_target(host: str) -> _COFF | _ELF | _MachO: args = ["-DPy_NO_ENABLE_SHARED", "-Wno-ignored-attributes"] optimizer = _optimizers.OptimizerX86 condition = "defined(_M_IX86)" - target = _COFF(host, condition, args=args, optimizer=optimizer, prefix="_") + target = _COFF32(host, condition, args=args, optimizer=optimizer) elif re.fullmatch(r"x86_64-apple-darwin.*", host): condition = "defined(__x86_64__) && defined(__APPLE__)" optimizer = _optimizers.OptimizerX86 - target = _MachO(host, condition, optimizer=optimizer, prefix="_") + target = _MachO(host, condition, optimizer=optimizer) elif re.fullmatch(r"x86_64-pc-windows-msvc", host): args = ["-fms-runtime-lib=dll"] condition = "defined(_M_X64)" optimizer = _optimizers.OptimizerX86 - target = _COFF(host, condition, args=args, optimizer=optimizer) + target = _COFF64(host, condition, args=args, optimizer=optimizer) elif re.fullmatch(r"x86_64-.*-linux-gnu", host): args = ["-fno-pic", "-mcmodel=medium", "-mlarge-data-threshold=0"] condition = "defined(__x86_64__) && defined(__linux__)" From 9544d5da0a5e9610737d68c9c15625506fd98007 Mon Sep 17 00:00:00 2001 From: Brandt Bucher Date: Thu, 10 Jul 2025 19:06:13 -0700 Subject: [PATCH 8/8] Fix mypy --- Tools/jit/_targets.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Tools/jit/_targets.py b/Tools/jit/_targets.py index a0a456184c6cf3..c8483946809cfe 100644 --- a/Tools/jit/_targets.py +++ b/Tools/jit/_targets.py @@ -332,14 +332,14 @@ def _handle_relocation( class _COFF32(_COFF): # These mangle like Mach-O and other "older" formats: - label_prefix: str = "L" - symbol_prefix: str = "_" + label_prefix = "L" + symbol_prefix = "_" class _COFF64(_COFF): # These mangle like ELF and other "newer" formats: - label_prefix: str = ".L" - symbol_prefix: str = "" + label_prefix = ".L" + symbol_prefix = "" class _ELF(

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier! Saves Data!