1
Fork 0

rustbuild: Update LLVM and enable ThinLTO

This commit updates LLVM to fix #45511 (https://reviews.llvm.org/D39981) and
also reenables ThinLTO for libtest now that we shouldn't hit #45768. This also
opportunistically enables ThinLTO for libstd which was previously blocked
(#45661) on test failures related to debuginfo with a presumed cause of #45511.

Closes #45511
This commit is contained in:
Alex Crichton 2017-11-15 05:16:23 -08:00
parent 85d50ce1c7
commit 47498dea09
4 changed files with 13 additions and 6 deletions

View file

@ -624,9 +624,7 @@ impl<'a> Builder<'a> {
cargo.arg("--release");
}
if mode != Mode::Libstd && // FIXME(#45320)
mode != Mode::Libtest && // FIXME(#45511)
self.config.rust_codegen_units.is_none() &&
if self.config.rust_codegen_units.is_none() &&
self.build.is_rust_llvm(compiler.host)
{
cargo.env("RUSTC_THINLTO", "1");

@ -1 +1 @@
Subproject commit f5532b22b5d741f3ea207b5b07e3e1ca63476f9b
Subproject commit 02b3734a5ba6de984eb5a02c50860cc014e58d56

@ -1 +1 @@
Subproject commit 51f104bf1cc6c3a588a11c90a3b4a4a18ee080ac
Subproject commit e45c75de1148456a9eb1a67c14a66df4dfb50c94

View file

@ -1,10 +1,19 @@
-include ../tools.mk
LOG := $(TMPDIR)/log.txt
# FIXME(#46126) ThinLTO for libstd broke this test
ifeq (1,0)
all:
ifeq ($(TARGET),x86_64-unknown-linux-gnu)
ifdef SANITIZER_SUPPORT
$(RUSTC) -C opt-level=1 -g -Z sanitizer=leak -Z print-link-args leak.rs | grep -q librustc_lsan
$(TMPDIR)/leak 2>&1 | grep -q 'detected memory leaks'
$(TMPDIR)/leak 2>&1 | tee $(LOG)
grep -q 'detected memory leaks' $(LOG)
endif
endif
else
all:
endif