Skip to content

clang 12.0.0#139

Merged
chrisburr merged 6 commits into
conda-forge:masterfrom
xhochy:12.0.0
May 17, 2021
Merged

clang 12.0.0#139
chrisburr merged 6 commits into
conda-forge:masterfrom
xhochy:12.0.0

Conversation

@xhochy

@xhochy xhochy commented May 11, 2021

Copy link
Copy Markdown
Member

Fixes #140

Checklist

  • Used a personal fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

@conda-forge-linter

Copy link
Copy Markdown

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.

Comment thread recipe/patches/0006-Fix-crosscompiling-LLVM-tools.patch Outdated
@xhochy

xhochy commented May 14, 2021

Copy link
Copy Markdown
Member Author

@isuruf OK to merge?

Comment thread recipe/build.sh
if [[ "$CONDA_BUILD_CROSS_COMPILATION" == "1" ]]; then
CMAKE_ARGS="${CMAKE_ARGS} -DLLVM_TABLEGEN_EXE=$BUILD_PREFIX/bin/llvm-tblgen -DNATIVE_LLVM_DIR=$BUILD_PREFIX/lib/cmake/llvm"
CMAKE_ARGS="${CMAKE_ARGS} -DCROSS_TOOLCHAIN_FLAGS_NATIVE=-DCMAKE_C_COMPILER=$CC_FOR_BUILD;-DCMAKE_CXX_COMPILER=$CXX_FOR_BUILD;-DCMAKE_C_FLAGS=-O2;-DCMAKE_CXX_FLAGS=-O2;-DCMAKE_EXE_LINKER_FLAGS=;-DCMAKE_MODULE_LINKER_FLAGS=;-DCMAKE_SHARED_LINKER_FLAGS=;-DCMAKE_STATIC_LINKER_FLAGS=;"
CMAKE_ARGS="${CMAKE_ARGS} -DCROSS_TOOLCHAIN_FLAGS_NATIVE=-DCMAKE_C_COMPILER=$CC_FOR_BUILD;-DCMAKE_CXX_COMPILER=$CXX_FOR_BUILD;-DCMAKE_C_FLAGS=-O2;-DCMAKE_CXX_FLAGS=-O2;-DCMAKE_EXE_LINKER_FLAGS=;-DCMAKE_MODULE_LINKER_FLAGS=;-DCMAKE_SHARED_LINKER_FLAGS=;-DCMAKE_STATIC_LINKER_FLAGS=;-DZLIB_ROOT=$BUILD_PREFIX"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

BUILD_PREFIX? why?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is needed for the cross-compiled LLVM tools to detect the correct libz. Otherwise, they also pick up the one in $PREFIX.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that this is part of the very long CROSS_TOOLCHAIN_FLAGS_NATIVE variable value.

@chrisburr chrisburr merged commit 2a8c361 into conda-forge:master May 17, 2021
@xhochy xhochy deleted the 12.0.0 branch May 17, 2021 18:44
@h-vetinari

Copy link
Copy Markdown
Member

osx-arm build timed out on master

@xhochy

xhochy commented May 18, 2021

Copy link
Copy Markdown
Member Author

Retriggered

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants