druidengine/druid

Move compiler flags from cmake files and into preset files.

Open

#21 aberto em 1 de out. de 2025

Ver no GitHub
 (0 comments) (0 reactions) (1 assignee)C++ (6 forks)auto 404
enhancementgood first issue

Métricas do repositório

Stars
 (3 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Is your feature request related to a problem? Please describe. I want to move the hard-coded compiler flags out of cmake configuration to allow users the flexibility to provide their own flags if they choose to not use cmake --preset and instead use cmake -DCMAKE_CXX_FLAGS=.

Describe the solution you'd like Currently, the compiler flags exist in cmake\project_compile_options.cmake. These flags need to be moved to their reespective use-<compiler>.json file found in cmake/preset/.

  • General compiler flags moved to:
    • use-clang.json
    • use-gcc.json
    • use-msvc.json
  • Create presets and move flags into:
    • use-clang-tsan.json. Inherits use-clang.
    • user-clang-asan.json. Inherits use-clang.
  • Create preset and move hardened flags into:
    • use-clang-hardened.json
    • use-gcc-hardened.json
    • use-msvc-hardened.json

Describe alternatives you've considered For the longest time, these flags remained in the cmake file to enfore all sub projects to use. With the advent of cmake presets this is no longer necessary to ensure quick, automatic, configuration.

Additional context I'm not 100% sure all the different compiler options work. They've been untested for a while.

Guia do colaborador

Direção de pesquisa
Estude a documentação de CMake presets e entenda como mover os flags do compilador de project compile options.cmake para os arquivos de preset use <compiler>.json. Certifique se de que a herança funcione corretamente para os presets sanitizer e hardened.
Pilha de tecnologia
cpp
Domain
build systemtooling
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++CMakecompiler flags
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60