dotnet/runtime

[mono] Use C11 atomics in atomic.h when building with MSVC

Aberta

#91.748 aberto em 7 de set. de 2023

Ver no GitHubGuia do colaborador
 (3 comentários) (0 reação) (0 responsável)C# (5.445 forks)batch import
area-Infrastructure-monohelp wanted

Métricas do repositório

Stars
 (17.886 estrelas)
Métricas de merge de PR
 (Mesclagem média 12d 11h) (661 fundiu PRs em 30d)

Description

There are two things we need to do:

  1. Pass the /experimental:c11atomics to the MSVC compiler for the runtime build
  2. Use C++23 when building our C++ files (they may include "atomic.h" which includes <stdatomic.h> which errors out in MSVC in C++ mode unless you're using at least C++23)

Guia do colaborador

Direção de pesquisa
Implemente as duas etapas: 1) Passe /experimental:c11atomics para o compilador MSVC na compilação do runtime. 2) Use C++23 para arquivos C++ que incluam atomic.h para evitar erros de compilação.
Pilha de tecnologia
ccpp
Domain
build systemcompilers
Tipo Issue
DevOps
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
CC++MSVCBuild systems
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70