In questo documento, riepiloghiamo le linee guida per esaminare le modifiche alla specifica. Al momento, queste modifiche in genere comportano la verifica di più cose da più fonti, quindi questo documento le riassume tutte per semplificare le revisioni:
- Verifica che la colonna "Specifica" in status.md indichi "yes". Aggiungi una riga se aggiungi una nuova operazione.
- Controlla se il titolo della sezione corrisponde alla mnemonica dell'operazione nell'ODS.
- Controlla se la sezione "Semantica" corrisponde alla Semantica dell'operazione di XLA.
- Verifica se le sezioni "Input" e "Output":
- Elenca le stesse voci dell'ODS.
- Elenca gli stessi elementi di HloInstruction::CreateFromProto.
- Vengono ordinate esattamente come gli ODS.
- In caso di mancate corrispondenze, verifica che siano presenti ticket corrispondenti.
- Controlla se la sezione "Vincoli":
- Corrisponde a shape_inference.cc di XLA.
- Corrisponde a hlo_verifier.cc di XLA.
- Corrisponde all'OTS.
- Corrisponde a StablehloOps.cpp.
- In caso di mancate corrispondenze, verifica che siano presenti ticket corrispondenti. Collega tutti i ticket nelle specifiche, in posizioni il più specifiche possibile (ad esempio, se un ticket riguarda un vincolo che non è stato implementato, collegalo direttamente in quel vincolo).
- Se le parti corrispondenti di ODS e StablehloOps.cpp corrispondono alle specifiche, controlla che le colonne "Verifica" e "Tipo di inferenza" in status.md riportino "sì".
- Verifica che la sezione "Esempi":
- Ha solo un esempio. In futuro, collegheremo ad altri esempi della suite di test per interpreti StableHLO.
- Utilizza una sintassi MLIR valida eseguendo
stablehlo-opt
su esempi di codice. - Utilizza una sintassi MLIR generica che può essere ottenuta eseguendo
stablehlo-opt -mlir-print-op-generic
(ci atteniamo a una sintassi generica nelle specifiche per evitare di dover modificare la specifica in caso di modifiche di piuttostoprinter).
- Verifica che il
description
nell'ODS dell'operazione:- Include la prima frase della specifica.
- Quindi si collega alla sezione corrispondente delle specifiche.
- Quindi utilizza lo stesso esempio della specifica, ma tramite una sintassi piuttosto semplice, che può essere ottenuta eseguendo
stablehlo-opt
.
- Controlla che i file relativi all'implementazione dei vincoli di verifica e di inferenza dei tipi siano conformi alle linee guida riportate di seguito:
- Segui la linea guida #1 per StablehloOps.td.
- Segui la linea guida #2 per TypeInference.cpp e StablehloOps.cpp.
- Segui la linea guida #5 per ops_stablehlo.mlir.
- Segui la linea guida #6 per infer_stablehlo.mlir.
- Valuta l'operatività per effetti collaterali e
speculabilità.
- Se l'operazione non ha effetti collaterali ed è sempre speculabile, assegna il trait
Pure
. Ciò è raro, poiché la maggior parte delle operazioni consente forme dinamiche, che possono portare a errate corrispondenze di forma in fase di runtime, che è un comportamento non definito. Alcune operazioni possono avere un comportamento indefinito anche in altre situazioni. La maggior parte delle operazioni non ha effetti collaterali (dovrebbe avere il trattoNoMemoryEffect
). - La maggior parte delle operazioni rientra in uno dei trait
HLO_SpeculatableIf*
. Se l'operazione non rientra in nessuno di questi, assegna il traitConditionallySpeculatable
e implementa i metodi dell'interfaccia. Aggiungi test astablehlo/tests/ops_speculatability.mlir
per coprire la logica di speculabilità.
- Se l'operazione non ha effetti collaterali ed è sempre speculabile, assegna il trait