llvm / llvm/llvm-project

AMDGPU: set FMINNUM_IEEE Legal while FMINNUM not

Open
#181,100 1 comment 0 reactions 0 assignees View on GitHub
backend:AMDGPU floating-point
Dominant language
LLVM
Stars
40.5k
Forks
18.7k
PR merge metrics
PR metrics pending

Description

FMINNUM_IEEE meets all requirement of FMINNUM.
So if we have FMINNUM_IEEE, we should also set FMINNUM to Legal.

Contributor guide

Open the contributing guide

Research direction

Start by locating the AMDGPU target legality handling for FMINNUM_IEEE and FMINNUM in the LLVM source. Check how the two operations are classified and confirm that FMINNUM becomes Legal wherever FMINNUM_IEEE is Legal. Done means the legality settings match the stated requirement and the relevant compiler tests pass.

Written by the indexing model from the issue text.

Assessment

Domain
compilers
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.