From 58b8acac0da5b14125c1ab5660e37748cb0b941e Mon Sep 17 00:00:00 2001 From: Catherine Date: Thu, 1 Jun 2023 18:17:44 +0000 Subject: [PATCH] _toolchain.cxx: remove. This is causing issues on Python 3.12 and in any case should be based on the Python `ziglang` package instead of this cursed setuptools hack. --- amaranth/_toolchain/cxx.py | 73 ------------------------------------- tests/test_toolchain_cxx.py | 68 ---------------------------------- 2 files changed, 141 deletions(-) delete mode 100644 amaranth/_toolchain/cxx.py delete mode 100644 tests/test_toolchain_cxx.py diff --git a/amaranth/_toolchain/cxx.py b/amaranth/_toolchain/cxx.py deleted file mode 100644 index a2b7006..0000000 --- a/amaranth/_toolchain/cxx.py +++ /dev/null @@ -1,73 +0,0 @@ -import tempfile -import sysconfig -import warnings -import os.path - - -__all__ = ["build_cxx"] - - -def build_cxx(*, cxx_sources, output_name, include_dirs, macros): - build_dir = tempfile.TemporaryDirectory(prefix="amaranth_cxx_") - - cwd = os.getcwd() - try: - # Unfortunately, `ccompiler.compile` assumes the paths are relative, and interprets - # the directory name of the source path specially. That makes it necessary to build in - # the output directory directly. - os.chdir(build_dir.name) - - with warnings.catch_warnings(): - warnings.filterwarnings(action="ignore", category=DeprecationWarning) - # This emits a DeprecationWarning on Python 3.10. - import setuptools - from distutils import ccompiler - cc_driver = ccompiler.new_compiler() - - cc_driver.output_dir = "." - - cc = sysconfig.get_config_var("CC") - cxx = sysconfig.get_config_var("CXX") - cflags = sysconfig.get_config_var("CCSHARED") - ld_flags = sysconfig.get_config_var("LDSHARED") - ld_cxxflags = sysconfig.get_config_var("LDCXXSHARED") - if ld_cxxflags is None: - # PyPy doesn't have LDCXXSHARED. Glue it together from CXX and LDSHARED and hope that - # the result actually works; not many good options here. - ld_cxxflags = " ".join([cxx.split()[0], *ld_flags.split()[1:]]) - cc_driver.set_executables( - compiler=f"{cc} {cflags}", - compiler_so=f"{cc} {cflags}", - compiler_cxx=f"{cxx} {cflags}", - linker_so=ld_cxxflags, - ) - - # Sometimes CCompiler is modified to have additional executable entries for compiling and - # linking CXX shared objects (e.g. on Gentoo). These executables have to be set then. - try: - cc_driver.set_executables( - compiler_so_cxx=f"{cxx} {cflags}", - linker_so_cxx=ld_cxxflags, - ) - except: - pass - - for include_dir in include_dirs: - cc_driver.add_include_dir(include_dir) - for macro in macros: - cc_driver.define_macro(macro) - for cxx_filename, cxx_source in cxx_sources.items(): - with open(cxx_filename, "w") as f: - f.write(cxx_source) - - cxx_filenames = list(cxx_sources.keys()) - obj_filenames = cc_driver.object_filenames(cxx_filenames) - so_filename = cc_driver.shared_object_filename(output_name) - - cc_driver.compile(cxx_filenames) - cc_driver.link_shared_object(obj_filenames, output_filename=so_filename, target_lang="c++") - - return build_dir, so_filename - - finally: - os.chdir(cwd) diff --git a/tests/test_toolchain_cxx.py b/tests/test_toolchain_cxx.py deleted file mode 100644 index 48f674e..0000000 --- a/tests/test_toolchain_cxx.py +++ /dev/null @@ -1,68 +0,0 @@ -import os -import ctypes -import tempfile -import unittest - -from amaranth._toolchain.cxx import * - - -class ToolchainCxxTestCase(unittest.TestCase): - def setUp(self): - self.include_dir = None - self.build_dir = None - - def tearDown(self): - if self.include_dir: - self.include_dir.cleanup() - if self.build_dir: - self.build_dir.cleanup() - - def test_filename(self): - self.build_dir, filename = build_cxx( - cxx_sources={"test.cc": ""}, - output_name="answer", - include_dirs=[], - macros=[], - ) - self.assertTrue(filename.startswith("answer")) - - def test_simple(self): - self.build_dir, filename = build_cxx( - cxx_sources={"test.cc": """ - extern "C" int answer() { return 42; } - """}, - output_name="answer", - include_dirs=[], - macros=[], - ) - library = ctypes.cdll.LoadLibrary(os.path.join(self.build_dir.name, filename)) - self.assertEqual(library.answer(), 42) - - def test_macro(self): - self.build_dir, filename = build_cxx( - cxx_sources={"test.cc": """ - extern "C" int answer() { return ANSWER; } - """}, - output_name="answer", - include_dirs=[], - macros=["ANSWER=42"], - ) - library = ctypes.cdll.LoadLibrary(os.path.join(self.build_dir.name, filename)) - self.assertEqual(library.answer(), 42) - - def test_include(self): - self.include_dir = tempfile.TemporaryDirectory(prefix="amaranth_hxx_") - with open(os.path.join(self.include_dir.name, "answer.h"), "w") as f: - f.write("#define ANSWER 42") - - self.build_dir, filename = build_cxx( - cxx_sources={"test.cc": """ - #include - extern "C" int answer() { return ANSWER; } - """}, - output_name="answer", - include_dirs=[self.include_dir.name], - macros=[], - ) - library = ctypes.cdll.LoadLibrary(os.path.join(self.build_dir.name, filename)) - self.assertEqual(library.answer(), 42)