Build servers with pre-GCC-5 ?

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Build servers with pre-GCC-5 ?

Hans Wennborg via cfe-dev
Hi,

We have some code that uses explicit std::move that is redundant with modern compilers. Are there any build servers that are still using pre-GCC-5 compilers? Could we just simply remove std::move without breaking any build server?
The minimum supported compiler version for LLVM/Clang seems to be GCC 5.1, but I'd like to get some confirmation from the community.

Thanks,
Gabor

_______________________________________________
cfe-dev mailing list
[hidden email]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev
Reply | Threaded
Open this post in threaded view
|

Re: Build servers with pre-GCC-5 ?

Hans Wennborg via cfe-dev

The minimum compiler version is enforced with a hard error in CMake.

 

-Eli

 

From: cfe-dev <[hidden email]> On Behalf Of Gábor Márton via cfe-dev
Sent: Friday, January 31, 2020 2:35 AM
To: Clang Dev <[hidden email]>
Subject: [EXT] [cfe-dev] Build servers with pre-GCC-5 ?

 

Hi,

 

We have some code that uses explicit std::move that is redundant with modern compilers. Are there any build servers that are still using pre-GCC-5 compilers? Could we just simply remove std::move without breaking any build server?

The minimum supported compiler version for LLVM/Clang seems to be GCC 5.1, but I'd like to get some confirmation from the community.

 

Thanks,

Gabor


_______________________________________________
cfe-dev mailing list
[hidden email]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev
Reply | Threaded
Open this post in threaded view
|

Re: Build servers with pre-GCC-5 ?

Hans Wennborg via cfe-dev
Hello Gabor,

I looked into this problem not so long ago, and my conclusion was that it is not gcc what is the limiting factor here -- clang is. Prior to clang-3.9, we were diagnosing code like:
struct A { };
struct B { B(A&&); };

B f(A a) { return a; }
----- 
Officially our minimum supported clang version is 3.5, so we need to use std::move(a) to keep this code building.

However, I don't know if we actually have any buildbots running with clang<3.9 to check that this is really the only thing that's needed to make those compilers work, or if we have anyone who cares about llvm buildability with those compilers (I certainly don't).

pl


On Fri, 31 Jan 2020 at 19:32, Eli Friedman via cfe-dev <[hidden email]> wrote:

The minimum compiler version is enforced with a hard error in CMake.

 

-Eli

 

From: cfe-dev <[hidden email]> On Behalf Of Gábor Márton via cfe-dev
Sent: Friday, January 31, 2020 2:35 AM
To: Clang Dev <[hidden email]>
Subject: [EXT] [cfe-dev] Build servers with pre-GCC-5 ?

 

Hi,

 

We have some code that uses explicit std::move that is redundant with modern compilers. Are there any build servers that are still using pre-GCC-5 compilers? Could we just simply remove std::move without breaking any build server?

The minimum supported compiler version for LLVM/Clang seems to be GCC 5.1, but I'd like to get some confirmation from the community.

 

Thanks,

Gabor

_______________________________________________
cfe-dev mailing list
[hidden email]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev

_______________________________________________
cfe-dev mailing list
[hidden email]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev