xenia-project/xenia

DIV_I32 and others don't handle INT_MIN / -1

Aperta

#1085 aperta il 12 feb 2018

Vedi su GitHubGuida contributor
 (2 commenti) (5 reazioni) (0 assegnatari)C++ (1077 fork)batch import
cpugood first issue

Metriche repository

Star
 (7418 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

https://github.com/benvanik/xenia/blob/190108dab6732d9db5c8595e1358b7b034cb5a14/src/xenia/cpu/backend/x64/x64_sequences.cc#L4332

It was pointed out to me that x86 will raise an exception when dividing INT_MIN by -1. I haven't encountered any games doing this yet - but it could potentially be an issue.

According to PowerPC docs, the output is undefined if this condition occurs.

Could be worth investigating if you're a new contributor.

Guida contributor

Direzione di ricerca
Indaga l'implementazione di DIV I32 in x64 sequences.cc per vedere se gestisce INT MIN / 1. Controlla il comportamento dell'eccezione x86 e la documentazione PowerPC per il comportamento indefinito. Proponi una correzione per evitare l'eccezione.
Tech stack
ccpp
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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++x86 assembly
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70