llvm/llvm-project

Enable exp10 libcall recognition on linux

Aberta

#64.677 aberto em 14 de ago. de 2023

Ver no GitHubGuia do colaborador
 (7 comentários) (0 reação) (2 responsáveis)C++ (10.782 forks)batch import
floating-pointgood first issuellvm:codegenllvm:optimizationsplatform:linux

Métricas do repositório

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

Description

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.

Guia do colaborador

Direção de pesquisa
A alteração está em llvm/lib/Analysis/TargetLibraryInfo.cpp por volta da linha 550. Remova a verificação de versão do glibc para exp10/exp10f/exp10l que foi adicionada devido a bugs no glibc <2.18. Garanta que a chamada de biblioteca seja reconhecida no Linux para glibc >=2.18 ou simplesmente remova a restrição por completo, conforme sugerido. Verifique se não há outras restrições de plataforma que precisem de ajuste.
Pilha de tecnologia
cpp
Domain
backend
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.
2
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
C++Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70