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

sys-libs/libcxx-9.0.0: fix error when compiling with clang+compiler-rt #13748

Closed
wants to merge 2 commits into from

Conversation

geraint0923
Copy link
Contributor

According to https://clang.llvm.org/docs/Toolchain.html#compiler-rt-llvm, -DLIBCXX_USE_COMPILER_RT=YES and -DLIBCXXABI_USE_COMPILER_RT=YES are explicitly needed to compile with clang+compiler-rt.

Otherwise libcxx will fail to build due to error like:
undefined reference to '__udivdi3'

@gentoo-bot gentoo-bot added assigned PR successfully assigned to the package maintainer(s). no bug found No Bug/Closes found in the commits. no signoff One or more commits do not indicate GCO sign-off. labels Nov 24, 2019
@geraint0923 geraint0923 changed the title sys-libs/libcxx-9.0.0: fix error when compiling with clang+compiler-rt sys-libs/libcxx-9.0.0: fix error when compiling with clang+compiler-rt [please reassign] Nov 24, 2019
@gentoo-bot gentoo-bot changed the title sys-libs/libcxx-9.0.0: fix error when compiling with clang+compiler-rt [please reassign] sys-libs/libcxx-9.0.0: fix error when compiling with clang+compiler-rt Nov 24, 2019
@gentoo-bot gentoo-bot added assigned PR successfully assigned to the package maintainer(s). no bug found No Bug/Closes found in the commits. and removed assigned PR successfully assigned to the package maintainer(s). no bug found No Bug/Closes found in the commits. no signoff One or more commits do not indicate GCO sign-off. labels Nov 24, 2019
@geraint0923 geraint0923 changed the title sys-libs/libcxx-9.0.0: fix error when compiling with clang+compiler-rt sys-libs/libcxx-9.0.0: fix error when compiling with clang+compiler-rt [please reassign] Nov 24, 2019
@gentoo-bot gentoo-bot changed the title sys-libs/libcxx-9.0.0: fix error when compiling with clang+compiler-rt [please reassign] sys-libs/libcxx-9.0.0: fix error when compiling with clang+compiler-rt Nov 24, 2019
@gentoo-bot gentoo-bot added assigned PR successfully assigned to the package maintainer(s). no bug found No Bug/Closes found in the commits. and removed assigned PR successfully assigned to the package maintainer(s). no bug found No Bug/Closes found in the commits. labels Nov 24, 2019
Copy link
Member

@mgorny mgorny left a comment

Choose a reason for hiding this comment

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

Nope, we aren't going to go all around the place adding flags for compiler-rt support. The ebuilds already have checks for compiler-rt being used, so just fix their results.

@geraint0923 geraint0923 force-pushed the fix_libcxx branch 3 times, most recently from bc1222a to 285593f Compare November 24, 2019 19:19
@geraint0923
Copy link
Contributor Author

Nope, we aren't going to go all around the place adding flags for compiler-rt support. The ebuilds already have checks for compiler-rt being used, so just fix their results.

Addressed the comment, please take a look.

According to
https://clang.llvm.org/docs/Toolchain.html#compiler-rt-llvm,
`-DLIBCXXABI_USE_COMPILER_RT=YES` is needed to linked against
compiler-rt if compiled with clang+compiler-rt.

Signed-off-by: Yang Yang <geraint0923@gmail.com>
According to
https://clang.llvm.org/docs/Toolchain.html#compiler-rt-llvm,
`-DLIBCXX_USE_COMPILER_RT=YES` is needed to linked to compiler-rt.

Otherwise building libcxx using clang+compiler-rt will fail.

Signed-off-by: Yang Yang <geraint0923@gmail.com>
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2019-11-24 21:42 UTC
Newest commit scanned: 687236a
Status: ✅ good

Issues already there before the PR (double-check them):
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#app-admin/consul
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#app-admin/consul-template
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#app-admin/puppet
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#app-admin/puppet-agent
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#app-admin/puppetdb
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#app-admin/puppetserver
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#app-admin/vault
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#app-antivirus/clamav-unofficial-sigs
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#app-emulation/docker
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#app-emulation/lxd
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#app-emulation/reg
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#app-metrics/blackbox_exporter
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#app-metrics/collectd
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#dev-db/mariadb
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#dev-db/pgbouncer
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#dev-db/postgresql
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#dev-lang/php
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#dev-lang/rust
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#dev-lang/rust-bin
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#dev-perl/MogileFS-Server
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#dev-php/pecl-http
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#dev-php/pecl-redis
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#dev-python/oslo-utils
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#dev-python/prometheus_client
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#dev-ruby/facter
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#dev-util/jenkins-bin
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#mail-filter/rspamd
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#media-libs/assimp
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#media-libs/coin
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#net-analyzer/nagios-core
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#net-analyzer/wireshark
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#net-dns/avahi
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#net-dns/bind
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#net-dns/unbound
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#net-mail/notmuch
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#net-misc/apt-cacher-ng
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#net-misc/dropbear
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#net-misc/gerbera
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#net-misc/memcached
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#net-misc/scponly
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#net-misc/stunnel
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#net-proxy/squid
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#net-vpn/tor
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#sys-apps/guix
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#sys-cluster/galera
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#sys-cluster/kube-apiserver
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#sys-cluster/kube-controller-manager
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#sys-cluster/kube-scheduler
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#sys-cluster/nova
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#sys-devel/binutils-hppa64
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#sys-process/cronie
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#www-apps/grafana-bin
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#www-apps/netbox
https://qa-reports.gentoo.org/output/gentoo-ci/894c17b/output.html#www-servers/nginx

@geraint0923 geraint0923 changed the title sys-libs/libcxx-9.0.0: fix error when compiling with clang+compiler-rt sys-libs/libcxx-9.0.0: fix error when compiling with clang+compiler-rt [please reassign] Nov 27, 2019
@gentoo-bot gentoo-bot changed the title sys-libs/libcxx-9.0.0: fix error when compiling with clang+compiler-rt [please reassign] sys-libs/libcxx-9.0.0: fix error when compiling with clang+compiler-rt Nov 27, 2019
@gentoo-bot
Copy link

Pull Request assignment

Submitter: @geraint0923
Areas affected: ebuilds
Packages affected: sys-libs/libcxx, sys-libs/libcxxabi

sys-libs/libcxx: aballier, @gentoo/llvm
sys-libs/libcxxabi: @gentoo/llvm

Linked bugs

No bugs to link found. If your pull request references any of the Gentoo bug reports, please add appropriate GLEP 66 tags to the commit message and request reassignment.

If you do not receive any reply to this pull request, please open or link a bug to attract the attention of maintainers.


In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

@gentoo-bot gentoo-bot added assigned PR successfully assigned to the package maintainer(s). no bug found No Bug/Closes found in the commits. and removed assigned PR successfully assigned to the package maintainer(s). no bug found No Bug/Closes found in the commits. labels Nov 27, 2019
Copy link
Member

@mgorny mgorny left a comment

Choose a reason for hiding this comment

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

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s). no bug found No Bug/Closes found in the commits.
Projects
None yet
4 participants