meilisearch/MeiliSearch

Enable LTO again

Ouverte

#2 718 ouverte le 29 août 2022

Voir sur GitHubGuide contributeur
 (7 commentaires) (1 réaction) (0 personne assignée)Rust (2 542 forks)batch import
enhancementhelp wantedperformancetech discussion

Métriques du dépôt

Stars
 (57 572 étoiles)
Métriques de merge PR
 (Merge moyen 11j 20h) (41 PRs mergées en 30 j)

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!

Guide contributeur

Direction de recherche
Recherchez comment activer conditionnellement LTO uniquement pour les cibles non aarch64 en Rust. Examinez les scripts de construction Cargo ou la compilation conditionnelle à l'aide d'attributs `cfg`. Envisagez d'utiliser `#[cfg(target arch = "aarch64")]` pour désactiver LTO pour cette architecture.
Stack technique
rust
Domaine
backend
Type d'issue
Performance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
RustGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65