clang-tool-chain 1.0.43__py3-none-any.whl → 1.0.44__py3-none-any.whl

clang_tool_chain/__version__.py

@@ -1,3 +1,3 @@
 """Version information for clang-tool-chain."""
 
-__version__ = "1.0.43"
+__version__ = "1.0.44"

clang_tool_chain/execution/arg_transformers.py

@@ -209,9 +209,10 @@ class LLDLinkerTransformer(ArgumentTransformer):
 
     Priority: 200 (runs after SDK but before ABI)
 
-    This transformer adds platform-specific linker flags:
-    - macOS: -fuse-ld=ld64.lld (explicit Mach-O linker, LLVM 21.x+)
-    - Linux: -fuse-ld=lld (standard ELF linker)
+    This transformer adds -fuse-ld=lld linker flag. The clang driver
+    automatically dispatches to the correct linker binary:
+    - macOS: ld64.lld (Mach-O linker)
+    - Linux: ld.lld (ELF linker)
 
     It also translates GNU ld flags to ld64.lld equivalents on macOS:
     - --no-undefined -> -undefined error
@@ -293,11 +294,16 @@ class ASANRuntimeTransformer(ArgumentTransformer):
     This transformer ensures proper ASAN runtime linking on Linux:
     - Detects -fsanitize=address flag
     - Adds -shared-libasan to use shared runtime library
+    - Adds -Wl,--allow-shlib-undefined when building shared libraries with ASAN
     - Prevents undefined symbol errors during linking
 
     The shared runtime library (libclang_rt.asan.so) contains the full
     ASAN implementation, while the static wrapper library only contains stubs.
 
+    When building shared libraries with sanitizers, the library may have undefined
+    symbols that will be provided by the sanitizer runtime when loaded. LLD by
+    default enforces no undefined symbols, so we need to allow them explicitly.
+
     Environment Variables:
         CLANG_TOOL_CHAIN_NO_SHARED_ASAN: Set to '1' to disable shared ASAN
     """
@@ -306,7 +312,9 @@ class ASANRuntimeTransformer(ArgumentTransformer):
         return 250
 
     def transform(self, args: list[str], context: ToolContext) -> list[str]:
-        """Add -shared-libasan when using ASAN on Linux."""
+        """Add -shared-libasan and --allow-shlib-undefined when using ASAN on Linux."""
+        import sys
+
         # Only applies to Linux clang/clang++
         if context.platform_name != "linux" or context.tool_name not in ("clang", "clang++"):
             return args
@@ -316,19 +324,41 @@ class ASANRuntimeTransformer(ArgumentTransformer):
         if not has_asan:
             return args
 
+        result = list(args)
+        injected_flags = []
+
         # Check if user disabled shared ASAN
-        if os.environ.get("CLANG_TOOL_CHAIN_NO_SHARED_ASAN") == "1":
+        if os.environ.get("CLANG_TOOL_CHAIN_NO_SHARED_ASAN") != "1":
+            # Check if -shared-libasan already present
+            if "-shared-libasan" not in args:
+                # Add -shared-libasan to use shared runtime library
+                # This prevents undefined symbol errors during linking
+                logger.info("Adding -shared-libasan for ASAN runtime linking on Linux")
+                result = ["-shared-libasan"] + result
+                injected_flags.append("-shared-libasan")
+        else:
             logger.debug("Shared ASAN disabled via CLANG_TOOL_CHAIN_NO_SHARED_ASAN=1")
-            return args
 
-        # Check if -shared-libasan already present
-        if "-shared-libasan" in args:
-            return args
+        # Check if building a shared library with ASAN
+        # Shared libraries need to allow undefined symbols that will be provided
+        # by the sanitizer runtime when the runner loads them
+        is_shared_lib = "-shared" in args
+        if is_shared_lib:
+            # Check if --allow-shlib-undefined already present
+            has_allow_shlib_undefined = any("--allow-shlib-undefined" in arg for arg in args)
+            if not has_allow_shlib_undefined:
+                logger.info("Adding -Wl,--allow-shlib-undefined for shared library with ASAN")
+                result = ["-Wl,--allow-shlib-undefined"] + result
+                injected_flags.append("-Wl,--allow-shlib-undefined")
+
+        # Warn on stderr if we injected flags
+        if injected_flags:
+            print(
+                f"clang-tool-chain: note: automatically injected sanitizer flags: {' '.join(injected_flags)}",
+                file=sys.stderr,
+            )
 
-        # Add -shared-libasan to use shared runtime library
-        # This prevents undefined symbol errors during linking
-        logger.info("Adding -shared-libasan for ASAN runtime linking on Linux")
-        return ["-shared-libasan"] + args
+        return result
 
 
 class RPathTransformer(ArgumentTransformer):

clang_tool_chain/linker/lld.py

@@ -35,11 +35,12 @@ def _llvm_supports_ld64_lld_flag() -> bool:
     """
     Check if the current platform's LLVM version supports -fuse-ld=ld64.lld.
 
-    The -fuse-ld=ld64.lld flag is only recognized by LLVM 21.x and later.
-    Earlier versions require -fuse-ld=lld (which auto-detects Mach-O from target).
+    Note: The clang driver does NOT recognize -fuse-ld=ld64.lld as a valid option.
+    This function exists for backward compatibility but always returns False in practice
+    because -fuse-ld=lld should be used instead (clang auto-dispatches to ld64.lld on Darwin).
 
     Returns:
-        True if LLVM >= 21.x, False otherwise.
+        True if LLVM >= 21.x (but we always fall back to -fuse-ld=lld anyway)
     """
     from ..platform.detection import get_platform_info
 
@@ -272,6 +273,39 @@ def _user_specified_lld_on_macos(args: list[str]) -> bool:
     return "-fuse-ld=lld" in args_str or "-fuse-ld=ld64.lld" in args_str
 
 
+def _user_specified_ld64_lld(args: list[str]) -> bool:
+    """
+    Check if the user explicitly specified -fuse-ld=ld64.lld.
+
+    This is used to emit a warning that the flag will be auto-converted
+    to -fuse-ld=lld since clang driver doesn't recognize ld64.lld.
+
+    Args:
+        args: Command-line arguments
+
+    Returns:
+        True if user specified -fuse-ld=ld64.lld
+    """
+    return any("-fuse-ld=ld64.lld" in arg for arg in args)
+
+
+def _convert_ld64_lld_to_lld(args: list[str]) -> list[str]:
+    """
+    Convert -fuse-ld=ld64.lld to -fuse-ld=lld in arguments.
+
+    The clang driver does not recognize -fuse-ld=ld64.lld as a valid option.
+    This function converts it to -fuse-ld=lld which the driver recognizes
+    and automatically dispatches to ld64.lld on Darwin targets.
+
+    Args:
+        args: Original compiler arguments
+
+    Returns:
+        Modified arguments with -fuse-ld=ld64.lld replaced by -fuse-ld=lld
+    """
+    return [arg.replace("-fuse-ld=ld64.lld", "-fuse-ld=lld") for arg in args]
+
+
 # pyright: reportUnusedFunction=false
 def _add_lld_linker_if_needed(platform_name: str, args: list[str]) -> list[str]:
     """
@@ -284,9 +318,10 @@ def _add_lld_linker_if_needed(platform_name: str, args: list[str]) -> list[str]:
     - Faster linking performance
     - Uniform toolchain across all platforms
 
-    Platform-specific linker flags:
-    - macOS: Uses -fuse-ld=ld64.lld (explicit Mach-O variant required by LLVM 21.x+)
-    - Linux: Uses -fuse-ld=lld (standard ELF linker)
+    Platform-specific behavior:
+    - Uses -fuse-ld=lld on all platforms (clang driver auto-dispatches)
+    - macOS: Clang driver finds ld64.lld (Mach-O linker)
+    - Linux: Clang driver finds ld.lld (ELF linker)
 
     The function is skipped when:
     - User sets CLANG_TOOL_CHAIN_USE_SYSTEM_LD=1
@@ -302,14 +337,25 @@ def _add_lld_linker_if_needed(platform_name: str, args: list[str]) -> list[str]:
         args: Original compiler arguments
 
     Returns:
-        Modified arguments with platform-specific -fuse-ld flag prepended if needed
+        Modified arguments with -fuse-ld=lld flag prepended if needed
     """
     # On macOS, if user explicitly specified LLD, we still need to translate flags
     # and ensure ld64.lld symlink exists, even though we don't inject the -fuse-ld flag ourselves
     if platform_name == "darwin" and _user_specified_lld_on_macos(args):
         logger.debug("User specified LLD on macOS, translating GNU ld flags to ld64.lld equivalents")
-        # Ensure ld64.lld symlink exists for user-specified -fuse-ld=ld64.lld
+        # Ensure ld64.lld symlink exists for lld to dispatch to Mach-O mode
         _ensure_ld64_lld_symlink()
+
+        # Check if user specified -fuse-ld=ld64.lld (which is not a valid clang driver option)
+        # and emit a warning about the auto-conversion to -fuse-ld=lld
+        if _user_specified_ld64_lld(args):
+            print(
+                "[clang-tool-chain] Warning: -fuse-ld=ld64.lld is not a valid clang driver option. "
+                "Auto-converting to -fuse-ld=lld (clang driver auto-dispatches to ld64.lld on Darwin).",
+                file=sys.stderr,
+            )
+            args = _convert_ld64_lld_to_lld(args)
+
         return _translate_linker_flags_for_macos_lld(args)
 
     if not _should_force_lld(platform_name, args):
@@ -320,22 +366,13 @@ def _add_lld_linker_if_needed(platform_name: str, args: list[str]) -> list[str]:
     # On macOS, translate GNU ld flags to ld64.lld equivalents
     if platform_name == "darwin":
         args = _translate_linker_flags_for_macos_lld(args)
+        # Ensure ld64.lld symlink exists for lld to dispatch to Mach-O mode
+        _ensure_ld64_lld_symlink()
 
-        # Check LLVM version to determine the correct linker flag
-        if _llvm_supports_ld64_lld_flag():
-            # LLVM 21.x+ supports explicit -fuse-ld=ld64.lld
-            _ensure_ld64_lld_symlink()
-            return ["-fuse-ld=ld64.lld"] + args
-        else:
-            # LLVM < 21.x: rewrite to -fuse-ld=lld with compatibility notice
-            version = _get_llvm_version_for_platform()
-            version_str = f"{version[0]}.{version[1]}.{version[2]}"
-            print(
-                f"[clang-tool-chain] LLVM {version_str} does not support -fuse-ld=ld64.lld, "
-                f"using -fuse-ld=lld instead (lld auto-detects Mach-O from target)",
-                file=sys.stderr,
-            )
-            return ["-fuse-ld=lld"] + args
-    else:
-        # Linux uses standard lld
-        return ["-fuse-ld=lld"] + args
+    # Always use -fuse-ld=lld on all platforms.
+    # Note: -fuse-ld=ld64.lld is NOT a valid clang driver option.
+    # The clang driver only recognizes generic names like "lld", "gold", "bfd".
+    # When -fuse-ld=lld is used, clang automatically dispatches to:
+    # - Darwin: ld64.lld (Mach-O linker)
+    # - Linux: ld.lld (ELF linker)
+    return ["-fuse-ld=lld"] + args

{clang_tool_chain-1.0.43.dist-info → clang_tool_chain-1.0.44.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: clang-tool-chain
-Version: 1.0.43
+Version: 1.0.44
 Summary: Clang Tool Chain - C/C++ compilation toolchain utilities
 Project-URL: Homepage, https://github.com/zackees/clang-tool-chain
 Project-URL: Repository, https://github.com/zackees/clang-tool-chain

{clang_tool_chain-1.0.43.dist-info → clang_tool_chain-1.0.44.dist-info}/RECORD

@@ -1,5 +1,5 @@
 clang_tool_chain/__init__.py,sha256=w3f1hdA72SH-WsfwesNTQbTeiXIlhV8ignySXrV3SfU,23
-clang_tool_chain/__version__.py,sha256=6gmxA-SFNIuF2QL2LcQn3fYERY3gD95uxieCn2hdE7o,72
+clang_tool_chain/__version__.py,sha256=VyBCcvXR6mMibUFhauoR0iyFq2jth9ReImF-qE_frUY,72
 clang_tool_chain/archive.py,sha256=t3reh7cm5XP2rhTqfRIDAQZv5XQq7SsstyiROYg8wFA,27697
 clang_tool_chain/archive_cache.py,sha256=5ZmlwXIJZDcrDFkTbdgBQYN9sulGn0WyI6qwWqC4HEU,6806
 clang_tool_chain/checksums.py,sha256=KFXeAeDz5ZlcZVOxsHDpNDCrm9UDoJ8bMA4PeNhuzdA,9868
@@ -37,7 +37,7 @@ clang_tool_chain/deployment/so_deployer.py,sha256=9mRKcJUXKV-8vXWJGdsUkB5cTPWjmT
 clang_tool_chain/directives/__init__.py,sha256=MJDNYL_MD2MF0HFsrTsSTX645bYo6vtjq2pOTtfykaU,198
 clang_tool_chain/directives/parser.py,sha256=6J7mO1JtvuHkkKS0Xges5b_jT9b3uTF6ULI0ZiwGAdw,11179
 clang_tool_chain/execution/__init__.py,sha256=wSYnlGmfTlaUPEKwW6PgwlxKTti6Wa5yyVjXVipdpcY,875
-clang_tool_chain/execution/arg_transformers.py,sha256=YkGqz_vbfNE-yFXHnzZREbS1T545AfQcV0DJTkU59YM,18254
+clang_tool_chain/execution/arg_transformers.py,sha256=vN-1T1mfD61Mg-P2B6hOQeLdD1RZA3l1pmNBbJSzo-g,19750
 clang_tool_chain/execution/build.py,sha256=PCtHw31WXbjCp2K_qaf1liaCiIjD49036cknqdQyhJM,13040
 clang_tool_chain/execution/build_pipeline.py,sha256=piqh7P2-GR0mVOik3OUGRFqWFIRr8LQM3PU7TcBsujk,17622
 clang_tool_chain/execution/core.py,sha256=7CJ0azznC5lq5bw8amk2kwCIN2I_OnDiKytpapkvrdY,25273
@@ -57,7 +57,7 @@ clang_tool_chain/installers/iwyu.py,sha256=9aAhdGtOTY6BrLuPtladY8Y2mz1i7FjgbMxZf
 clang_tool_chain/installers/lldb.py,sha256=FpG8NMNQk8PoNfg6aeU_plmSQrVET7zo-pTvoK8z838,2261
 clang_tool_chain/installers/nodejs.py,sha256=5N07rotgmCfUaDm1uJfBlIAFKC1iTpgZT0HBRuoYwKI,9343
 clang_tool_chain/linker/__init__.py,sha256=ghzDFpZ2-gPmdDO6K05C7yNbY6pZLANPuUks9TaQwVY,537
-clang_tool_chain/linker/lld.py,sha256=J8XXtz0DaabnNFReSJCXgk37NloUvrIkjp5TzIZ7Tg8,12468
+clang_tool_chain/linker/lld.py,sha256=08qkvoK4-8yACm34GfZzp3vM8rZYHCv_M9fjUeSetd0,13826
 clang_tool_chain/platform/__init__.py,sha256=WkV9Y25ua0mtzEGcsIxF-qExtroSTAMKkcElWuQF2BE,342
 clang_tool_chain/platform/detection.py,sha256=PLHyUfmQ5xuohhpz0KSXJWK3d0u0fCsjx1DbM8f1CxQ,5470
 clang_tool_chain/platform/paths.py,sha256=K0IjeVwbmgPlAWQO8mS3r1WS4C2dN6IYrSqPpckeT5c,6088
@@ -67,8 +67,8 @@ clang_tool_chain/sdk/windows.py,sha256=8zMLAoFz2OKMz-w6Kqxr3A-6Cofto2VWZvCvRi7kn
 clang_tool_chain/testing/__init__.py,sha256=-sYqOOCuTV_u-MkmExrD4uKdTHG4RmMwR3D1kIG281Q,208
 clang_tool_chain/testing/diagnostic_runner.py,sha256=mnmFUEOQulY3-Ggu6hKVGZwjrKQNmV6kY80PRTUu2qU,5293
 clang_tool_chain/testing/diagnostic_tests.py,sha256=GmtKWrDcddZTpx9_yIKfhRAy6YOde8dj7SksCWVEME4,6019
-clang_tool_chain-1.0.43.dist-info/METADATA,sha256=BEMrWFJ8AIyAAdlb5v0CfKWkfKr1I-BNYkzoNAv2H6w,50425
-clang_tool_chain-1.0.43.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
-clang_tool_chain-1.0.43.dist-info/entry_points.txt,sha256=DBfnqZJzHFoO6rldGY3ESaX-z2ydAvIueUPM9ss9M-w,2728
-clang_tool_chain-1.0.43.dist-info/licenses/LICENSE,sha256=51FO1oc2pZbQNI0v0_THnznnZIF4iFgawG1xnQ58kKo,10997
-clang_tool_chain-1.0.43.dist-info/RECORD,,
+clang_tool_chain-1.0.44.dist-info/METADATA,sha256=SjvdkUZlqBEHGRSpfaWoAys6sGSSOkH3_b9BdvKFOd0,50425
+clang_tool_chain-1.0.44.dist-info/WHEEL,sha256=WLgqFyCfm_KASv4WHyYy0P3pM_m7J5L9k2skdKLirC8,87
+clang_tool_chain-1.0.44.dist-info/entry_points.txt,sha256=DBfnqZJzHFoO6rldGY3ESaX-z2ydAvIueUPM9ss9M-w,2728
+clang_tool_chain-1.0.44.dist-info/licenses/LICENSE,sha256=51FO1oc2pZbQNI0v0_THnznnZIF4iFgawG1xnQ58kKo,10997
+clang_tool_chain-1.0.44.dist-info/RECORD,,