llvm/llvm-project

Enable exp10 libcall recognition on linux

Open

#64.677 aperta il 14 ago 2023

Vedi su GitHub
 (7 commenti) (0 reazioni) (2 assegnatari)C++ (10.782 fork)batch import
floating-pointgood first issuellvm:codegenllvm:optimizationsplatform:linux

Metriche repository

Star
 (26.378 star)
Metriche merge PR
 (Merge medio 1g 2h) (1000 PR mergiate in 30 g)

Descrizione

TargetLibraryInfo does not report exp10/exp10f/exp10l as available on Linux, disabling all special library call optimizations:

https://github.com/llvm/llvm-project/blob/c0682840877a21960f4c21a97b5f497d4ccc82d5/llvm/lib/Analysis/TargetLibraryInfo.cpp#L550

The reason cited is these were buggy in glibc prior to version 2.18. Version 2.18 celebrated its 10th birthday this weekend, so I think it's alright to just drop this restriction.

Guida contributor

Direzione di ricerca
La modifica si trova in llvm/lib/Analysis/TargetLibraryInfo.cpp intorno alla riga 550. Rimuovere il controllo della versione di glibc per exp10/exp10f/exp10l che è stato aggiunto a causa di bug in glibc <2.18. Assicurarsi che la chiamata di libreria venga riconosciuta su Linux per glibc >=2.18 o semplicemente rimuovere del tutto la restrizione come suggerito. Verificare che non ci siano altri vincoli di piattaforma da regolare.
Tech stack
cpp
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70