llvm/llvm-project

AMDGPU should implement IR version of isFMAFasterThanFMulAndFAdd

Open

#108.751 aperta il 15 set 2024

Vedi su GitHub
 (12 commenti) (0 reazioni) (1 assegnatario)C++ (10.782 fork)batch import
backend:AMDGPUgood first issuemissed-optimization

Metriche repository

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

Descrizione

TargetLowering has an IR variant of isFMAFasterThanFMulAndFAdd, and AMDGPU does not yet implement it.

As far as I can tell, this is only ever used in target code to implement isProfitableToHoist (e.g. AArch64 here https://github.com/llvm/llvm-project/blob/3ae71d154e5dfb5e5a5d27b3699b27ce2b55f44d/llvm/lib/Target/AArch64/AArch64ISelLowering.cpp#L16086)

So for this to be effective, isProfitableToHoist would also need to be implemented similarly.

Guida contributor

Direzione di ricerca
Studia l'implementazione esistente di isFMAFasterThanFMulAndFAdd in AArch64 (in AArch64ISelLowering.cpp) e la variante IR in TargetLowering. Comprendi le caratteristiche hardware FMA di AMDGPU. Quindi implementa la versione IR per AMDGPU in AMDGPUISelLowering.cpp. Implementa anche isProfitableToHoist in modo simile. Testa con i test lit pertinenti.
Tech stack
ccpp
Dominio
compilers
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
4
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++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
35