Alexander_KS/SAPFOR
4
2
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

cmake: set -O2 optimization level only if CMAKE_BUILD_TYPE is not specified #66

Merged
Alexander_KS merged 2 commits from cmake_linux_opt_level into master 2025-09-25 18:13:00 +00:00
Conversation 3 Commits 2 Files Changed 1 +4 -2
xnpster commented 2025-09-25 10:53:22 +00:00
Collaborator
Copy Link

Сейчас для linux всегда ставится уровень оптимизации -O2, независимо от CMAKE_BUILD_TYPE
Это ведёт к следующим проблемам:

  • при сборке Debug включаются O2 оптимизации, что мешает отладке
  • при сборке Release используется O2, а не O3
  • флаг O2 и флаги от типа сборки перемешиваются: например, при сборке Release получается что-то вроде -std=c++17 -O2 -O3 -DNDEBUG, что не очень хорошо

Решение: устанавливать O2 только если CMAKE_BUILD_TYPE не определён

Сейчас для linux всегда ставится уровень оптимизации -O2, независимо от CMAKE_BUILD_TYPE Это ведёт к следующим проблемам: - при сборке Debug включаются O2 оптимизации, что мешает отладке - при сборке Release используется O2, а не O3 - флаг O2 и флаги от типа сборки перемешиваются: например, при сборке Release получается что-то вроде `-std=c++17 -O2 -O3 -DNDEBUG`, что не очень хорошо Решение: устанавливать O2 только если CMAKE_BUILD_TYPE не определён
xnpster added 1 commit 2025-09-25 10:53:22 +00:00
cmake: set -O2 optimization level only if CMAKE_BUILD_TYPE is not specified c7a31c8bbb
Alexander_KS commented 2025-09-25 17:30:25 +00:00
Owner
Copy Link

а зачем тогда оставлять этот флаг, если в релизе O3, тогда надо удалить эту строчку вовсе.

а зачем тогда оставлять этот флаг, если в релизе O3, тогда надо удалить эту строчку вовсе.
xnpster commented 2025-09-25 17:33:37 +00:00
Author
Collaborator
Copy Link

Чтобы в случае, если CMAKE_BUILD_TYPE не указан, выставлялся хотя бы -O2

Чтобы в случае, если CMAKE_BUILD_TYPE не указан, выставлялся хотя бы -O2
Alexander_KS commented 2025-09-25 17:36:20 +00:00
Owner
Copy Link

так если он не указал, вставляется O3 - разве нет? надо тогда разделять на дебаг - релиз. Наверно это лишнее все таки.

так если он не указал, вставляется O3 - разве нет? надо тогда разделять на дебаг - релиз. Наверно это лишнее все таки.
xnpster added 1 commit 2025-09-25 17:49:28 +00:00
cmake: change default optimization level to O3 9c0fd40e7b
Alexander_KS merged commit cad0a0b217 into master 2025-09-25 18:13:00 +00:00
Alexander_KS deleted branch cmake_linux_opt_level 2025-09-25 18:13:00 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
abrosim0v Dudarenko EgorMayorov gr_gusev IgorBobrov oleg_nikitin xnpster Alexander_KS
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Alexander_KS/SAPFOR#66
Delete Branch "cmake_linux_opt_level"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?