Page MenuHomeFreeBSD

Remove tests for obsolete compilers in the build system

Authored by vangyzen on May 10 2020, 10:09 PM.



Assume gcc is at least 6.4, the oldest xtoolchain in the ports tree.
Assume clang is at least 6, which was in 11.2-RELEASE. Drop conditions
for older compilers.

Test Plan
  1. tinderbox with in-tree toolchain
  2. amd64 buildworld/buildkernel with amd64-gcc xtoolchain (6.4.0)
  3. amd64 buildworld/buildkernel on 11.2-RELEASE

(There were unrelated failures in 1 and 2.)

Diff Detail

rS FreeBSD src repository - subversion
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

74 ↗(On Diff #71625)

I just realized I can completely remove these three conditions.

  • remove unnecessary if-clang conditions

overall LGTM

217 ↗(On Diff #71627)

do we need this conditional at all?

217 ↗(On Diff #71627)

I'm glad you asked. I don't know. Are there other compilers capable of building FreeBSD?

Intel released a FreeBSD version of their 2016 compiler. I intend to try building FreeBSD with it (in my copious spare time).

Follow on work may include assuming c++11 or c++14, but this is a great start and should be committed as is, module units from the review.

142 ↗(On Diff #71627)

We should note that we should clean up warnings produced with these flags. They were originally there as a quick hack to enable gcc5/6

217 ↗(On Diff #71627)

We don't need it. Intel compiler support needs to be based on someone using it successfully. Changes to support it are likely more extensive than this one line...

46 ↗(On Diff #71627)

Do we still need this?

36 ↗(On Diff #71627)


This revision is now accepted and ready to land.May 10 2020, 10:59 PM
32 ↗(On Diff #71627)

Shouldn't this -Wno option be removed instead ? I do not even see this warning option (either with or without no-) in gcc 10.1 manual.

32 ↗(On Diff #71627)

The test was not GCC-before-4.3 so I think the patch is what's intended?
Perhaps we need to separately check this option though.

vangyzen added inline comments.
32 ↗(On Diff #71627)

It is still needed for in-tree clang, at least.

217 ↗(On Diff #71627)

I love the deadpan delivery of this understatement.

46 ↗(On Diff #71627)

Apparently not.

This revision now requires review to proceed.May 11 2020, 3:22 PM
This revision is now accepted and ready to land.May 11 2020, 3:26 PM
jhb added inline comments.
84 ↗(On Diff #71636)

Can't this be removed since we only support armv6 and armv7 now? If so, that probably should be a separate commit though.