Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add patches for PPC #39094

Merged
merged 5 commits into from
Jan 13, 2021
Merged

add patches for PPC #39094

merged 5 commits into from
Jan 13, 2021

Conversation

vchuravy
Copy link
Member

@vchuravy vchuravy commented Jan 4, 2021

@vchuravy vchuravy added backport 1.6 Change should be backported to release-1.6 external dependencies Involves LLVM, OpenBLAS, or other linked libraries system:powerpc PowerPC labels Jan 4, 2021
@wal004

This comment has been minimized.

vchuravy and others added 2 commits January 12, 2021 17:41
GCC 11 has changed header dependencies again, so the build of LLVM
fails because <limits> wasn't being included when it should be. This
is already fixed upstream in LLVM 12.
@vchuravy vchuravy marked this pull request as ready for review January 12, 2021 22:57
@@ -39,7 +39,7 @@ endef
# If $(2) == `src`, this will generate a `USE_BINARYBUILDER_FOO=0` make flag
# It will also generate a `FOO_BB_TRIPLET=$(2)` make flag.
define make_flags
USE_BINARYBUILDER=$(if $(filter src,$(2)),0,1) $(call makevar,$(1))_BB_TRIPLET=$(if $(filter src,$(2)),,$(2)) LLVM_ASSERTIONS=$(if $(filter assert,$(3)),1,0) DEPS_GIT=0
USE_BINARYBUILDER=$(if $(filter src,$(2)),0,1) $(call makevar,$(1))_BB_TRIPLET=$(if $(filter src,$(2)),,$(2)) BINARYBUILDER_LLVM_ASSERTS=$(if $(filter assert,$(3)),1,0) DEPS_GIT=0
Copy link
Member Author

Choose a reason for hiding this comment

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

Will be changed back in #38775

@vchuravy vchuravy merged commit 70db77a into master Jan 13, 2021
@vchuravy vchuravy deleted the vc/ppc_llvmpatches branch January 13, 2021 22:19
@KristofferC KristofferC mentioned this pull request Jan 19, 2021
60 tasks
@KristofferC KristofferC removed the backport 1.6 Change should be backported to release-1.6 label Jan 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external dependencies Involves LLVM, OpenBLAS, or other linked libraries system:powerpc PowerPC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Power] Segmentation fault in vecelement test
4 participants