conda-forge / llvmdev-feedstock

A conda-smithy repository for llvmdev.
BSD 3-Clause "New" or "Revised" License
8 stars 41 forks source link

llvmdev 16.0.0.rc4 #201

Closed h-vetinari closed 1 year ago

conda-forge-webservices[bot] commented 1 year ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

github-actions[bot] commented 1 year ago

Hi! This is the friendly conda-forge automerge bot!

I considered the following status checks when analyzing this PR:

Thus the PR was not passing and not merged.

h-vetinari commented 1 year ago

Interesting, something is causing ninja install in install_llvm.sh to repeat the full build from build.sh.

I remember that this happened before in clangdev, where this was caused/fixed by:

@isuruf: I marked as clang_bootstrap 11.rc builds as broken to avoid different compilers in top level build vs output level build envs which caused make to recompile.

This would seem to point to the merge of https://github.com/conda-forge/clang-compiler-activation-feedstock/pull/91 as the culprit, but the compilers are actually the same between build.sh and the libllvm16 output:

build.sh:

    clang_bootstrap_osx-64: 14.0.6-h6ddedbb_4         conda-forge
    libcxx:                 14.0.6-hccf4f1f_0         conda-forge

libllvm16:

    clang_bootstrap_osx-64: 14.0.6-h6ddedbb_4         conda-forge
    libcxx:                 15.0.7-h71dddab_0         conda-forge

I'm not 100% if that's causing it, but what did stand out was a difference in the libcxx being pulled in (https://github.com/conda-forge/libcxx-feedstock/pull/108 was merged after #198).

@isuruf, if you have an idea what's going on here, your input would be much appreciated.

isuruf commented 1 year ago

Did you run a verbose ninja build?

h-vetinari commented 1 year ago

So ninja --verbose install doesn't give much more info (aside from spelling out the full call), it just starts recompiling immediately:

+ cd build
+ ninja --verbose install
[1/3315] /Users/runner/miniforge3/conda-bld/llvm-package_1678664749514/_build_env/bin/arm64-apple-darwin20.0.0-clang++ -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -I/Users/runner/miniforge3/conda-bld/llvm-package_1678664749514/work/build/lib/Demangle -I/Users/runner/miniforge3/conda-bld/llvm-package_1678664749514/work/llvm/lib/Demangle -I/Users/runner/miniforge3/conda-bld/llvm-package_1678664749514/work/build/include -I/Users/runner/miniforge3/conda-bld/llvm-package_1678664749514/work/llvm/include -ftree-vectorize -fPIC -fPIE -fstack-protector-strong -O2 -pipe -stdlib=libc++ -fvisibility-inlines-hidden -fmessage-length=0 -isystem /Users/runner/miniforge3/conda-bld/llvm-package_1678664749514/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/include -fdebug-prefix-map=/Users/runner/miniforge3/conda-bld/llvm-package_1678664749514/work=/usr/local...
[2/3315] /Users/runner/miniforge3/conda-bld/llvm-package_1678664749514/_build_env/bin/arm64-apple-darwin20.0.0-clang++ -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -I/Users/runner/miniforge3/conda-bld/llvm-package_1678664749514/work/build/lib/Demangle -I/Users/runner/miniforge3/conda-bld/llvm-package_1678664749514/work/llvm/lib/Demangle -I/Users/runner/miniforge3/conda-bld/llvm-package_1678664749514/work/build/include -I/Users/runner/miniforge3/conda-bld/llvm-package_1678664749514/work/llvm/include -ftree-vectorize -fPIC -fPIE -fstack-protector-strong -O2 -pipe -stdlib=libc++ -fvisibility-inlines-hidden -fmessage-length=0 -isystem /Users/runner/miniforge3/conda-bld/llvm-package_1678664749514/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placeh/include -fdebug-prefix-map=/Users/runner/miniforge3/conda-bld/llvm-package_1678664749514/work=/usr/local...

(note I didn't shorten the lines; the ... at the end is how they appear in azure for osx-arm)