Skip to content

Commit

Permalink
Revert "Use default system compiler version on Travis (#22820)" (#23856)
Browse files Browse the repository at this point in the history
This reverts commit 2641555.

GCC 4.8 miscompiles our LLVM pass.

Fix #23839
  • Loading branch information
yuyichao authored and JeffBezanson committed Sep 24, 2017
1 parent 440a8a7 commit 0588ba4
Showing 1 changed file with 21 additions and 7 deletions.
28 changes: 21 additions & 7 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,29 +5,37 @@ matrix:
include:
- os: linux
env: ARCH="i686"
compiler: "g++ -m32"
compiler: "g++-5 -m32"
addons:
apt:
sources:
- ubuntu-toolchain-r-test
packages:
- libssl1.0.0
- bar
- time
- gcc-multilib
- g++-multilib
- binutils
- gcc-5
- g++-5
- gcc-5-multilib
- g++-5-multilib
- make:i386
- libssl-dev:i386
- gfortran
- gfortran-multilib
- gfortran-5
- gfortran-5-multilib
- os: linux
env: ARCH="x86_64"
compiler: "g++ -m64"
compiler: "g++-5 -m64"
addons:
apt:
sources:
- ubuntu-toolchain-r-test
packages:
- libssl1.0.0
- bar
- time
- gfortran
- g++-5
- gfortran-5
- os: osx
env: ARCH="x86_64"
osx_image: xcode8
Expand Down Expand Up @@ -56,6 +64,12 @@ before_install:
- make check-whitespace
- if [ `uname` = "Linux" ]; then
contrib/travis_fastfail.sh || exit 1;
mkdir -p $HOME/bin;
ln -s /usr/bin/gcc-5 $HOME/bin/gcc;
ln -s /usr/bin/g++-5 $HOME/bin/g++;
ln -s /usr/bin/gfortran-5 $HOME/bin/gfortran;
ln -s /usr/bin/gcc-5 $HOME/bin/x86_64-linux-gnu-gcc;
ln -s /usr/bin/g++-5 $HOME/bin/x86_64-linux-gnu-g++;
gcc --version;
BAR="bar -i 30";
BUILDOPTS="-j5 VERBOSE=1 FORCE_ASSERTIONS=1 LLVM_ASSERTIONS=1";
Expand Down

2 comments on commit 0588ba4

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

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

Executing the daily benchmark build, I will reply here when finished:

@nanosoldier runbenchmarks(ALL, isdaily = true)

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

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

Your benchmark job has completed - possible performance regressions were detected. A full report can be found here. cc @ararslan

Please sign in to comment.