This is an archive of the discontinued LLVM Phabricator instance.

Add support for detection of devtoolset-8
ClosedPublic

Authored by rpopescu on Mar 29 2019, 3:58 AM.

Details

Summary

The current llvm/clang et al. project can be built with the latest developer toolset (devtoolset-8) on RHEL, which provides GCC 8.2.1.
However, the result compiler will not identify this toolset itself when compiling programs, which is of course not desirable.

After the patch - which simply adds the name of the developer toolset to the existing list - it gets identified and selected, as shown below:

[bamboo@bamboo llvm-project]$ clang -v
clang version 9.0.0 (https://github.com/llvm/llvm-project.git e5ac385fb1ffa4bd3875ea6a4d24efdbd7814572)
Target: x86_64-unknown-linux-gnu
Thread model: posix
InstalledDir: /home/bamboo/llvm/bin
Found candidate GCC installation: /opt/rh/devtoolset-4/root/usr/lib/gcc/x86_64-redhat-linux/5.2.1
Found candidate GCC installation: /opt/rh/devtoolset-7/root/usr/lib/gcc/x86_64-redhat-linux/7
Found candidate GCC installation: /opt/rh/devtoolset-8/root/usr/lib/gcc/x86_64-redhat-linux/8
Found candidate GCC installation: /usr/lib/gcc/x86_64-redhat-linux/4.8.2
Found candidate GCC installation: /usr/lib/gcc/x86_64-redhat-linux/4.8.5
Selected GCC installation: /opt/rh/devtoolset-8/root/usr/lib/gcc/x86_64-redhat-linux/8
Candidate multilib: .;@m64
Candidate multilib: 32;@m32
Selected multilib: .;@m64

Diff Detail

Repository
rL LLVM

Event Timeline

rpopescu created this revision.Mar 29 2019, 3:58 AM
tstellar accepted this revision.Mar 29 2019, 6:24 AM

LGTM.

This revision is now accepted and ready to land.Mar 29 2019, 6:24 AM

Do you have commit access?

Do you have commit access?

Hi Tom, I don't think that I do. I have created the account just before submitting the patch. Should I request it, or?

Do you have commit access?

Hi Tom, I don't think that I do. I have created the account just before submitting the patch. Should I request it, or?

I'll commit it for you.

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptApr 9 2019, 6:24 AM