meilisearch/MeiliSearch

Enable LTO again

Aberta

#2.718 aberto em 29 de ago. de 2022

Ver no GitHubGuia do colaborador
 (7 comentários) (1 reação) (0 responsável)Rust (2.542 forks)batch import
enhancementhelp wantedperformancetech discussion

Métricas do repositório

Stars
 (57.572 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

We first enabled LTO in this PR https://github.com/meilisearch/meilisearch/pull/2690 to improve the performance of the search engine.

Then, we had to disable LTO in this PR https://github.com/meilisearch/meilisearch/pull/2717 because we were not able to compile the aarch64 binary anymore. See: https://github.com/meilisearch/meilisearch/actions/runs/2948932141

We would introduce LTO again and prevent using it when compiling the aarch64 binary. Some beginning of solution here

If anyone has a better solution, we would happy to read you!

Guia do colaborador

Direção de pesquisa
Pesquise como habilitar condicionalmente LTO apenas para targets não aarch64 em Rust. Veja scripts de build do Cargo ou compilação condicional usando atributos `cfg`. Considere usar `#[cfg(target arch = "aarch64")]` para desabilitar LTO para essa arquitetura.
Pilha de tecnologia
rust
Domain
backend
Tipo Issue
Performance
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.
Meio dia
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
RustGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65