[test-suite] r263456 - cmake/caches: Add some comments, more aggressive release mode

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 14 11:46:56 PDT 2016




> On Mar 14, 2016, at 11:33 AM, Quentin Colombet via llvm-commits <llvm-commits at lists.llvm.org> wrote:
> 
> Hi Matthias,
> 
>> On Mar 14, 2016, at 11:12 AM, Matthias Braun via llvm-commits <llvm-commits at lists.llvm.org> wrote:
>> 
>> Author: matze
>> Date: Mon Mar 14 13:12:36 2016
>> New Revision: 263456
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=263456&view=rev
>> Log:
>> cmake/caches: Add some comments, more aggressive release mode
>> 
>> Added:
>>   test-suite/trunk/cmake/caches/Release_Haswell.cmake
>> Modified:
>>   test-suite/trunk/cmake/caches/Debug.cmake
>>   test-suite/trunk/cmake/caches/Release.cmake
>> 
>> Modified: test-suite/trunk/cmake/caches/Debug.cmake
>> URL: http://llvm.org/viewvc/llvm-project/test-suite/trunk/cmake/caches/Debug.cmake?rev=263456&r1=263455&r2=263456&view=diff
>> ==============================================================================
>> --- test-suite/trunk/cmake/caches/Debug.cmake (original)
>> +++ test-suite/trunk/cmake/caches/Debug.cmake Mon Mar 14 13:12:36 2016
>> @@ -1,3 +1,5 @@
>> +# Lowest optimization level and maximum debug information; A typical developer
>> +# configuration.
>> +set(CMAKE_BUILD_TYPE "Debug" CACHE STRING "")
>> set(CMAKE_C_FLAGS_DEBUG "-O0 -g" CACHE STRING "")
>> set(CMAKE_CXX_FLAGS_DEBUG "-O0 -g" CACHE STRING "")
>> -set(CMAKE_BUILD_TYPE "Debug" CACHE STRING "")
>> 
>> Modified: test-suite/trunk/cmake/caches/Release.cmake
>> URL: http://llvm.org/viewvc/llvm-project/test-suite/trunk/cmake/caches/Release.cmake?rev=263456&r1=263455&r2=263456&view=diff
>> ==============================================================================
>> --- test-suite/trunk/cmake/caches/Release.cmake (original)
>> +++ test-suite/trunk/cmake/caches/Release.cmake Mon Mar 14 13:12:36 2016
>> @@ -1,4 +1,9 @@
>> -set(CMAKE_C_FLAGS_RELEASE "-O3 -flto" CACHE STRING "")
>> -set(CMAKE_CXX_FLAGS_RELEASE "-O3 -flto" CACHE STRING "")
>> +# Aggressive settings to produce fast code.
>> +set(OPTFLAGS "${OPTFLAGS} -O3 -march=native -fomit-frame-pointer -flto -DNDEBUG”)
> 
> I may be completely off as I actually don’t know when Release.cmake is used, but I am not sure I want the "-march=native” to be applied silently. The rationale is that we possibly cannot share our binary anymore between different CPUs.
This stuff is still in early experimental phases and not used by any in-production bots yet AFAIK. But the plan is to put common bot configurations there so it gets easy to reproduce a given build and indeed -march=XXX is a bad idea for reproducability across machines. In fact I realized that too and created Release_Haswell.cmake after ponding a bit over it and then forgot to remove the -march=native from the Release.cmake :)

- Matthias

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160314/cc89c48c/attachment.html>


More information about the llvm-commits mailing list