Skip to content

Commit b7ccb0a

Browse files
committed
Auto merge of #45810 - SimonSapin:ac-dc, r=aturon
Disable LLVM assertions on Nightly, enable them in "alt" builds. Per IRC discussion https://mozilla.logbot.info/rust-infra/20171106#c13812170-c13812204 Background: https://internals.rust-lang.org/t/disabling-llvm-assertions-in-nightly-builds/5388/14
2 parents e312c8a + 7625c79 commit b7ccb0a

File tree

4 files changed

+9
-7
lines changed

4 files changed

+9
-7
lines changed

.travis.yml

+3-3
Original file line numberDiff line numberDiff line change
@@ -16,14 +16,14 @@ matrix:
1616
if: type = pull_request OR branch = auto
1717

1818
- env: IMAGE=dist-x86_64-linux DEPLOY=1
19-
if: branch = try OR branch = auto
19+
if: branch = auto
2020

2121
# "alternate" deployments, these are "nightlies" but don't have assertions
2222
# turned on, they're deployed to a different location primarily for projects
2323
# which are stuck on nightly and don't want llvm assertions in the artifacts
2424
# that they use.
2525
- env: IMAGE=dist-x86_64-linux DEPLOY_ALT=1
26-
if: branch = auto
26+
if: branch = try OR branch = auto
2727

2828
- env: >
2929
RUST_CHECK_TARGET=dist
@@ -322,7 +322,7 @@ deploy:
322322
secure: "kUGd3t7JcVWFESgIlzvsM8viZgCA9Encs3creW0xLJaLSeI1iVjlJK4h/2/nO6y224AFrh/GUfsNr4/4AlxPuYb8OU5oC5Lv+Ff2JiRDYtuNpyQSKAQp+bRYytWMtrmhja91h118Mbm90cUfcLPwkdiINgJNTXhPKg5Cqu3VYn0="
323323
on:
324324
branch: try
325-
condition: $DEPLOY = 1
325+
condition: $DEPLOY_ALT = 1
326326

327327
# this is the same as the above deployment provider except that it uploads to
328328
# a slightly different directory and has a different trigger

src/bootstrap/config.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -528,7 +528,7 @@ impl Config {
528528
// Now that we've reached the end of our configuration, infer the
529529
// default values for all options that we haven't otherwise stored yet.
530530

531-
let default = config.channel == "nightly";
531+
let default = false;
532532
config.llvm_assertions = llvm_assertions.unwrap_or(default);
533533

534534
let default = match &config.channel[..] {

src/ci/run.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ if [ "$DEPLOY$DEPLOY_ALT" != "" ]; then
4949
if [ "$NO_LLVM_ASSERTIONS" = "1" ]; then
5050
RUST_CONFIGURE_ARGS="$RUST_CONFIGURE_ARGS --disable-llvm-assertions"
5151
elif [ "$DEPLOY_ALT" != "" ]; then
52-
RUST_CONFIGURE_ARGS="$RUST_CONFIGURE_ARGS --disable-llvm-assertions"
52+
RUST_CONFIGURE_ARGS="$RUST_CONFIGURE_ARGS --enable-llvm-assertions"
5353
fi
5454
else
5555
# We almost always want debug assertions enabled, but sometimes this takes too

src/test/run-make/sanitizer-memory/Makefile

+4-2
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,10 @@
33
all:
44
ifeq ($(TARGET),x86_64-unknown-linux-gnu)
55
ifdef SANITIZER_SUPPORT
6-
$(RUSTC) -g -Z sanitizer=memory -Z print-link-args uninit.rs | grep -q librustc_msan
7-
$(TMPDIR)/uninit 2>&1 | grep -q use-of-uninitialized-value
6+
$(RUSTC) -g -Z sanitizer=memory -Z print-link-args uninit.rs | tee $(TMPDIR)/out
7+
grep -q librustc_msan $(TMPDIR)/out
8+
$(TMPDIR)/uninit 2>&1 | tee $(TMPDIR)/out
9+
grep -q use-of-uninitialized-value $(TMPDIR)/out
810
endif
911
endif
1012

0 commit comments

Comments
 (0)