Discussion :

[Seabirds]

Bonjour à toutes et à tous !

Quand je compile mon script avec cette ligne:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

g++ -Wall -o3 main_1.cpp -std=c++17 -DNDEBUG -DBOOST_UBLAS_NDEBUG -I/usr/include/gdal  -L/usr/lib/ -lgdal -I/home/becheler/dev

Tout se passe bien, et ça tourne en quelques secondes.

Mais quand j'essaie de faire la même avec un CMakeLists.txt, baaaah, ça compile mais ça met une plombe à l'exécution. Je dois mal m'y prendre avec cmake parce que l'option -DBOOST_UBLAS_NDEBUG semble ne pas être prise en compte (j'ai de bonnes raisons de croire que l'énorme ralentissement du code que j'ai proviens de là). J'ai essayé de bidouiller en tentant différents trucs mais ... fail !

Pourriez-vous m'aider je vous prie ? Voici mon CMakeLists.txt actuel:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
 
cmake_minimum_required(VERSION 2.8)
 
project(quetzal)
 
add_compile_options(-Wall -pedantic-errors -Wextra -std=c++17 -ofast)
 
FIND_PACKAGE(Boost)
IF (Boost_FOUND)
    INCLUDE_DIRECTORIES(${Boost_INCLUDE_DIR})
    ADD_DEFINITIONS( "-DHAS_BOOST -DBOOST_UBLAS_NDEBUG" ) # ça semble pas marcher
ENDIF()
 
# j'ai aussi essayé
# add_definitions(-DBOOST_UBLAS_NDEBUG) #sans plus de succès
 
FIND_PACKAGE(GDAL)
IF (GDAL_FOUND)
    INCLUDE_DIRECTORIES(${GDAL_INCLUDE_DIR})
ENDIF()
 
include_directories( ./ )
 
add_executable(expansion_1 documentation/code_examples/expansion/main_1.cpp)
TARGET_LINK_LIBRARIES(expansion_1 LINK_PUBLIC ${GDAL_LIBRARY} )

Merci !

[Bousk]

Ça n'a peut-être rien à voir mais pourquoi les quotes autour des définitions ?
https://cmake.org/cmake/help/v3.8/co...finitions.html

j'ai de bonnes raisons de croire que l'énorme ralentissement du code que j'ai proviens de là

Quelles sont ces raisons ?
L'exe compilé a-t-il une taille significativement différente ?

[Seabirds]

J'ai enlevé les guillemets, cela n'a rien changé.

Quelles sont ces raisons ?
L'exe compilé a-t-il une taille significativement différente ?

Compilé avec Cmake, l'exé a une taille de 230.9 kB
Compilé a la main, l'exé a une taille de 1.2 MB.

Je faisais pour ma part plutôt allusion a d'anciennes galères d’exécution de programme interminable, qui avait été résolu avec avec cette fameuse option de compilation -DBOOST_UBLAS_NDEBUG.
L'explication vaut ce qu'elle vaut ...

[bacelar]

C'est pas la version "-g" que vous générez via Cmake ???

[Seabirds]

Salut bacelar

Si par la tu entends que je n'ai pas rajoute l'option -DNDEBUG, c'est parce que j'appelle cmake avec

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
cmake -DCMAKE_BUILD_TYPE=Release ..
make

Ce qui normalement devrait rajouter l'option -DNDEBUG, non ?

[jo_link_noir]

Pas sûr que tu compares les bons exécutables. Sur la ligne de commande de gcc il y a -o3 et dans le CmakeLists -ofast. Or, -o est l'option pour nommé le fichier de sortie, celle pour les optimisations est en majuscule: -O.

Tu peux faire make VERBOSE=1 pour avoir les lignes de compilation, et voir s'il manque vraiment -DHAS_BOOST -DBOOST_UBLAS_NDEBUG.

-DCMAKE_BUILD_TYPE=Release active bien NDEBUG. Pour bien faire, il faudrait aussi que -Ofast ne s'active qu'en Realease, par défaut ce doit être un -O2.

Il faut utiliser set(CMAKE_CXX_STANDARD 17) à la place de -std=c++17. Sans, cmake peut définir une autre version du standard.

[Seabirds]

Pas sûr que tu compares les bons exécutables. Sur la ligne de commande de gcc il y a -o3 et dans le CmakeLists -ofast. Or, -o est l'option pour nommé le fichier de sortie, celle pour les optimisations est en majuscule: -O.

Ah ouais. Omg. Violent. Beuah. Ca fait juste 3 ans que je renomme mes fichiers au lieu de les optimiser. La grande claaasse.
En tout cas merci d'avoir relevé cette mécompréhension typographique, en vrai je trouvais curieux que l'option d'optimisation change aussi le nom de l'exécutable. Prrrrffff je m'auto-gonfle.

Tu peux faire make VERBOSE=1 pour avoir les lignes de compilation, et voir s'il manque vraiment -DHAS_BOOST -DBOOST_UBLAS_NDEBUG.

Merci je vais faire ça !

-DCMAKE_BUILD_TYPE=Release active bien NDEBUG. Pour bien faire, il faudrait aussi que -Ofast ne s'active qu'en Realease, par défaut ce doit être un -O2.

Ah oui pas bête !

Il faut utiliser set(CMAKE_CXX_STANDARD 17) à la place de -std=c++17. Sans, cmake peut définir une autre version du standard.

"Définir une autre version du standard"... Cette phrase me laisse décidément songeur... Mais ok je fais ça !

[Seabirds]

Bon, de retour du front.
J'ai suivi les conseils de jo_link_noir, et après quelques bidouilles pour faire coincider les lignes de compilation, je suis arrivé à faire tourner la bestiole. Ce que je trouve bizarre, c'est qu'activer une option d'optimisation fait buguer (ou ralentir) le programme
Un avis là dessus ?

CMakeLists.txt :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
cmake_minimum_required(VERSION 2.8)
 
project(quetzal)
 
set(CMAKE_CXX_STANDARD 17)
add_compile_options(-Wall -pedantic-errors -Wextra -DNDEBUG)
 
FIND_PACKAGE(Boost)
IF (Boost_FOUND)
    INCLUDE_DIRECTORIES(${Boost_INCLUDE_DIR})
    ADD_DEFINITIONS(-DBOOST_UBLAS_NDEBUG)
ENDIF()
 
FIND_PACKAGE(GDAL)
IF (GDAL_FOUND)
    INCLUDE_DIRECTORIES(${GDAL_INCLUDE_DIR})
ENDIF()
 
include_directories( ./ )
 
add_executable(expansion_1 documentation/code_examples/expansion/main_1.cpp)
TARGET_LINK_LIBRARIES(expansion_1 LINK_PUBLIC ${GDAL_LIBRARY} )

Un bout de la sortie du cmake VERBOSE=1 :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
/usr/bin/c++  -DBOOST_UBLAS_NDEBUG -I/usr/include/gdal -I/home/becheler/dev/quetzal/.  -Wall -pedantic-errors -Wextra -DNDEBUG -std=gnu++1z -o CMakeFiles/expansion_1.dir/documentation/code_examples/expansion/main_1.cpp.o -c /home/becheler/dev/quetzal/documentation/code_examples/expansion/main_1.cpp

[jo_link_noir]

Étrange, il n'y a aucune option d'optimisation dans ce que tu donnes. Tu es bien dans un dossier initialisé avec cmake -DCMAKE_BUILD_TYPE=Release ? Ou alors les options par défaut sont à la ramasse .

Tu peux regarder dans le fichier CMakeCache.txt les options associées à chaque type de build et au besoin les modifier.

[ternel]

Commence par utiliser cmake avec une version moins périmée, donc, exige au moins la "3.0"
Auquel cas, tu auras accès aux définitions de propriétés. Ca te sera utile lorsque tu commenceras à écrire des bibliothèques.

Code cmake :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
cmake_minimum_required(VERSION 3.8) # première version à supporter C++17
 
project(quetzal)
 
find_package(Boost)
find_package(GDAL)
 
# force all compilation in this project with those options.
add_compile_options(-Wall -pedantic-errors -Wextra)
 
add_executable(expansion_1 documentation/code_examples/expansion/main_1.cpp)
set_target_properties(expansion_1 PROPERTIES CXX_EXTENSIONS OFF) # prefer iso standards to compiler extensions ones.
target_compile_features(expansion_1 PUBLIC cxx_std_17)
 
target_include_directories(expansion_1 PRIVATE .)
 
if(Boost_FOUND)
  target_include_directories(expansion_1 PRIVATE ${Boost_INCLUDE_DIR})
  target_compile_definitions(expansion_1 PRIVATE BOOST_UBLAS_NDEBUG)
endif()
 
if(GDAL_FOUND)
  target_include_directories(expansion_1 PRIVATE ${GDAL_INCLUDE_DIR})
  target_link_libraries(expansion_1 PUBLIC ${GDAL_LIBRARY} )
endif()

Ensuite, je t'invite à lire la documentation sur les "CMAKE_BUILD_TYPE", dont la valeur par défaut est "aucun réglages"
CMAKE_BUILD_TYPE=Release permet de rajouter des options "-O3 -DNDEBUG" (en général).

Et n'hésite pas à utiliser ccmake pour éditer ton cache ou même faire ta génération.

[Katian]

Il faut utiliser set(CMAKE_CXX_STANDARD 17) à la place de -std=c++17. Sans, cmake peut définir une autre version du standard.

merci pour l'astuce ! ^^

[koala01]

Salut,

Juste un truc:

Quand tu places la ligne

Code cmake :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
<snip>
add_compile_options(-Wall -pedantic-errors -Wextra)
<snip>

Tu forces l'utilisation d'options spécifiques à Gcc (et, heureusement supportées par clang) quel que soit le compilateur considéré.

Ce n'est pas catastrophique tant que tu n'envisage d'utiliser que ces deux compilateurs, mais ce sont des options qui risquent d'être rejetées en bloc par VC++, pour ne citer que celui là

De plus, cela fait déjà plusieurs versions de Gcc que l'on conseille d'utiliser l'option --all-warnings à la place de -Wall et l'option --extra-warnings à la place de -WextraJe sais bien que Wall et Wextra sont les options "historiques", mais, comme on ne peut pas garantir qu'elles ne seront pas supprimées dans le futur (du simple fait qu'il existe une alternative plus récente), il me semble préférable d'utiliser --all-warnings et --extra-warnings

[Pyramidev]

De plus, cela fait déjà plusieurs versions de Gcc que l'on conseille d'utiliser l'option --all-warnings à la place de -Wall et l'option --extra-warnings à la place de -WextraJe sais bien que Wall et Wextra sont les options "historiques", mais, comme on ne peut pas garantir qu'elles ne seront pas supprimées dans le futur (du simple fait qu'il existe une alternative plus récente), il me semble préférable d'utiliser --all-warnings et --extra-warnings

Dans la documentation de GCC 8.2 sur les avertissements, je ne vois ni --all-warnings, ni --extra-warnings.

[koala01]

Faut les cherchr un peu

Ma version de Gcc:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
gcc -v
Utilisation des specs internes.
COLLECT_GCC=gcc
COLLECT_LTO_WRAPPER=/usr/lib/gcc/x86_64-linux-gnu/8/lto-wrapper
OFFLOAD_TARGET_NAMES=nvptx-none
OFFLOAD_TARGET_DEFAULT=1
Cible : x86_64-linux-gnu
Configuré avec: ../src/configure -v --with-pkgversion='Debian 8.2.0-9' --with-bugurl=file:///usr/share/doc/gcc-8/README.Bugs --enable-languages=c,ada,c++,go,brig,d,fortran,objc,obj-c++ --prefix=/usr --with-gcc-major-version-only --program-suffix=-8 --program-prefix=x86_64-linux-gnu- --enable-shared --enable-linker-build-id --libexecdir=/usr/lib --without-included-gettext --enable-threads=posix --libdir=/usr/lib --enable-nls --with-sysroot=/ --enable-clocale=gnu --enable-libstdcxx-debug --enable-libstdcxx-time=yes --with-default-libstdcxx-abi=new --enable-gnu-unique-object --disable-vtable-verify --enable-libmpx --enable-plugin --enable-default-pie --with-system-zlib --with-target-system-zlib --enable-objc-gc=auto --enable-multiarch --disable-werror --with-arch-32=i686 --with-abi=m64 --with-multilib-list=m32,m64,mx32 --enable-multilib --with-tune=generic --enable-offload-targets=nvptx-none --without-cuda-driver --enable-checking=release --build=x86_64-linux-gnu --host=x86_64-linux-gnu --target=x86_64-linux-gnu
Modèle de thread: posix
gcc version 8.2.0 (Debian 8.2.0-9)

Demandons l'aide au sujet des warnings

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
gcc --help=warnings
Les options suivantes contrôlent les messages d’avertissement du compilateur:
  --all-warnings              Comme -Wall. Utilisez plutôt cette dernière
                              option.
  --extra-warnings            Comme -Wextra. Utilisez plutôt cette dernière
                              option.
  -W                          This switch is deprecated; use -Wextra instead. 
                              Comme -Wextra.
(...)
  -Wextra                     Afficher des avertissements complémentaires (au
                              risque qu'ils soient non désirés).

Bon, ce n'est pas dans la doc officielle (qui manque de mise à jour, de toute évidence), mais, comme c'est dans l'aide officielle du programme, on devrait pouvoir s'y fier, non?
[/CODE]

[jo_link_noir]

--all-warnings Comme -Wall. Utilisez plutôt cette dernière option.

La dernière fait référence à -Wall.

S'il faut les chercher c'est bien parce que personne ne les recommande, rien que dans les derniers changelogs il y a plusieurs références à -Wall. Ce qui est plutôt logique puisque les options de gcc sont avec un seul tiret et les avertissements préfixés de W. --all-warnings et --extra-warnings sont des extraterrestres là-dedans.