Technical discussions | Discussions techniques
|
aCOSwt, Friday 19 April 2013 à 22:33
|
|
Subscription date : 07 October 2012
Messages : 22
|
Bonjour,
Je crains qu'il n'y ait un petit bug dans les CMakelists.txt du core et des plugins lignes 17,18,19
if (NOT ${CMAKE_BUILD_TYPE})
add_definitions (-O3)
endif()
En fait, cela ne fait pas grand chose que forcer systématiquement l'optimisation de gcc à -03 quelle que soit la valeur de CMAKE_BUILD_TYPE
Pour que ce code soit efficace (et c'est effectivement intéressant que de conditionner le forçage au setting du CMAKE_BUILD_TYPE), il faut, à mon avis faire :
if (NOT CMAKE_BUILD_TYPE)
add_definitions (-O3)
endif()
(CMAKE_BUILD_TYPE sans dollar ni accolades)
En fait... c'est un peu plus qu'à mon avis... j'ai testé et ça marche mieux comme cela. 
Merci. |
|
matttbe, Friday 19 April 2013 à 23:39
|
|
Subscription date : 24 January 2009
Messages : 12573
|
Hello,
En effet, bien vu! (je pensais que les lignes du début venaient toutes de la doc, je n'aurais pas pensé à regardé!)
C'est fixé sur BZR! |
Subscription date : 30 November 2007
Messages : 17118
|
du coup est-ce qu'on utilisait l'option dans nos paquets ou pas ? |
|
matttbe, Sunday 21 April 2013 à 19:50
|
|
Subscription date : 24 January 2009
Messages : 12573
|
Pour les paquets deb, il semblerait que CMAKE_BUILD_TYPE ne soit pas utilisé. |
Subscription date : 30 November 2007
Messages : 17118
|
donc pas de 03 ? c'est dommage de se passer des optimisations |
|
matttbe, Tuesday 23 April 2013 à 00:18
|
|
Subscription date : 24 January 2009
Messages : 12573
|
Ils utilisent un -O2 je pense (je suppose qu'ils ont leurs raisons), ce qui n'est pas très différents du -O3 et aussi, ils splittent les symboles de debug du reste. |
Technical discussions | Discussions techniques
|
Français
Online users :