depends: libtapi build still builds a version 6.0.1 ClangBasic target #19540

issue fanquake opened this issue on July 17, 2020
  1. fanquake commented at 2:52 AM on July 17, 2020: member

    The libtapi we build in depends builds a ClangBasic target. This is currently LLVM Clang 6.0.1, as that repository has a copy of Apples swift-clang repository at this commit: https://github.com/apple/swift-clang/commit/3c8cb332b1fed69f67aecfd56675b0ff297f2a01. You'll also see in our depends build output:

    -- Looking for sys/resource.h
    -- Looking for sys/resource.h - found
    -- Clang version: 6.0.1
    -- Performing Test CXX_SUPPORTS_NO_NESTED_ANON_TYPES_FLAG
    -- Performing Test CXX_SUPPORTS_NO_NESTED_ANON_TYPES_FLAG - Success
    -- TAPI version: 10.0.0
    -- Supported Architectures: i386;x86_64;x86_64h;armv4t;armv6;armv5;armv7;armv7s;armv7k;armv6m;armv7m;armv7em;arm64
    -- Performing Test LINKER_SUPPORTS_NO_INITS
    -- Performing Test LINKER_SUPPORTS_NO_INITS - Failed
    -- Configuring done
    -- Generating done
    -- Build files have been written to: /bitcoin/depends/work/build/x86_64-apple-darwin16/native_cctools/55562e4073dea0fbfd0b20e0bf69ffe6390c7f97-8eb3b80cace/libtapi/build
    
    ## Building clangBasic ##
    ...
    

    Given that we've just moved to Clang 8.0.0 in depends for macOS builds, and generally we like to keep all these things "in sync", we should check if anything needs to be done here. Updating this would require upstream to reintegrate it's repo with a newer LLVM, which should be https://github.com/apple/llvm-project, given Apple has deprecated swift-clang.

    Opening an issue to remind myself but if anyone else wants to take a look please do.

  2. fanquake added the label Build system on Jul 17, 2020
  3. fanquake added the label Upstream on Jul 17, 2020
  4. fanquake commented at 12:53 PM on August 25, 2020: member

    Upstream has now been updated, and is using Clang 8.

  5. theuni commented at 8:47 PM on August 26, 2020: member

    Agree with keeping in sync, and concept ACK to bumping whenever this is ready upstream.

    One thing to keep in mind, though, is that libtapi is only used to link .tbd stubs from our SDK, which is pinned. So technically it shouldn't matter which version of clang files we link against as long as the result is capable of linking our pre-determined SDK files. This would be much more likely to cause issues at the time of an SDK bump, if for example, the newer SDK's .tbd files used a newer syntax.

  6. fanquake closed this on Mar 18, 2021

  7. DrahtBot locked this on Aug 18, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-17 00:14 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me