llvm/llvm-project

[InstCombiner] foldIntrinsicIsFPClass - incorrect submasking of FPClassTest masks

Open

#189.949 aberto em 1 de abr. de 2026

Ver no GitHub
 (15 comments) (0 reactions) (1 assignee)C++ (10.782 forks)batch import
floating-pointgood first issuellvm:instcombine

Métricas do repositório

Stars
 (26.378 stars)
Métricas de merge de PR
 (Mesclagem média 1d 2h) (1.000 fundiu PRs em 30d)

Description

As mentioned here when creating the DAG equivalent: https://github.com/llvm/llvm-project/pull/189944#discussion_r3021637130

We shouldn't be looking for equality, we should be testing that they are a subset of the known classes:

https://github.com/llvm/llvm-project/blob/03b5539dd58584e6821e11123d5c8546147a6f41/llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp#L1180-L1184

Guia do colaborador

Direção de pesquisa
Revise o código no local referenciado e altere a condição de verificação de igualdade (==) para uma verificação de subconjunto usando operações bit a bit, como `(Mask & Known) == Mask`.
Pilha de tecnologia
cpp
Domain
backend
Tipo Issue
Bug
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.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50