Dalam dokumen ini, kami merangkum panduan untuk mengimplementasikan dan meninjau operasi bagi penafsir. Kami sengaja menyertakan beberapa item tindakan tambahan yang terkait dengan verifier dan inferensi jenis, dengan ide untuk membuat progres di bidang tersebut bersama dengan implementasi penafsir.
Saat menerapkan operasi
- Berikan strategi pengujian yang ditulis secara eksplisit (dalam deskripsi PR) yang serupa dengan ini untuk digunakan sebagai referensi saat meninjau metode inferensi verifikasi dan jenis, serta pengujian yang terkait. Peninjau akan memeriksa kembali apakah deskripsinya komprehensif.
- Lihat hlo_evaluator untuk mengidentifikasi detail implementasi yang rumit dan potensi kesenjangan fungsi.
- Ajukan tiket untuk komponen software yang sesuai jika Anda menemukan bug atau fungsi yang tidak ada.
Setelah menerapkan operasi
Di StablehloOps.td:
- Pastikan
summary
dalam ODS operasi mengikuti format standar. (tiket terkait) Tambahkan komentar yang mereferensikan label batasan (misalnya,
Cn
atauIn
) dari spec dalam formatxyz_cn
atauxyz_in
, untuk opXyzOp
, guna mengidentifikasi korespondensi antara batasan di ODS dan spesifikasi. Contoh berikut menunjukkan cara menambahkan label batasan sebagai komentar bersama mlirTraits
danTypeConstraints
. Catatanxyz_c4
mengacu pada batasan yang ditentukan dalam classStableHLO_FooOp
(misalnyaStableHLO_ShapedInterfaceOp
,StableHLO_UnaryElementwiseOp
,StableHLO_Op
, dll.).def StableHLO_XyzOp: StableHLO_FooOp<"xyz", [Trait1, Trait2 /*xyz_c1, xyz_c2*/, InferTensorType /*xyz_c3*/]> { /*xyz_c4*/ ... let summary = "Xyz operation"; let arguments = (ins 1DTensorOf<[HLO_Float]>:$a, /*xyz_c5, xyz_i1*/ HLO_Tensor:$b, /*xyz_i2*/ .... ); );
- Pastikan
Di TypeInference.cpp dan StablehloOps.cpp:
- Hapus komentar yang berisi hal-hal seperti "Verifikasi properti berikut: ...".
- Tambahkan komentar yang mereferensikan label batasan (misalnya
Cn
atauIn
) dari spec dalam formatxyz_cn
atauxyz_in
, untuk opXyzOp
, guna mengidentifikasi bagian verifier dan fungsi bentuk yang sesuai dengan batasan dalam spesifikasi.- Anda dapat memiliki komentar dengan beberapa label batasan atau memiliki
beberapa komentar dengan label batasan yang sama. Semuanya bergantung pada
cara batasan diterapkan. Jika ada batasan berturut-turut,
ringkas batasan tersebut sebagai
xyz_cn...xyz_cm, xyz_in...xyz_jn
. - Jika ada ketidakcocokan antara batasan dalam implementasi VS dan batasan dalam spesifikasi, pastikan ada masalah terbuka yang mencerminkan perbedaan tersebut.
- Anda dapat memiliki komentar dengan beberapa label batasan atau memiliki
beberapa komentar dengan label batasan yang sama. Semuanya bergantung pada
cara batasan diterapkan. Jika ada batasan berturut-turut,
ringkas batasan tersebut sebagai
Dalam pengujian penafsir:
- Tambahkan file bernama
<op_mnemonic>.mlir
. - Tulis pengujian dengan mengikuti panduan pengujian.
- Tambahkan file bernama
-
- Jalankan pengujian yang dinonaktifkan yang tercakup dalam operasi yang baru ditambahkan.
- Jika lulus pengujian, aktifkan pengujian dengan mengonversi
RUN-DISABLED
menjadiRUN
. - Jika pengujian gagal karena alasan selain ketidakcocokan presisi, perbaiki implementasi/pengujian.
- Untuk ketidakcocokan presisi, beri tag pada pengujian dengan
RUN-DISABLED(#1278)
(jika belum dilakukan).
-
- Pastikan ada setidaknya satu pengujian (positif atau negatif) untuk setiap batasan dalam metode verifier dan inferensi jenis; batasan yang tercakup dalam ODS tidak akan diuji. Pengujian ini sebagian besar akan bersifat negatif, yang menguji bahwa batasan tidak terpenuhi atau positif, yang menguji bahwa bentuk yang disimpulkan sudah benar.
- Pastikan semua pengujian yang terkait dengan operasi yang sedang diuji ditempatkan bersama.
- Pastikan semua pengujian yang terkait dengan operasi yang sedang diuji
diawali dengan makro lit
CHECK-LABEL
. - Pilih nama fungsi pengujian menggunakan format
xyz_cn_im_...
untuk batasan pengujian fungsiCn
,Im
, dll. untuk opXyzOp
. Jika format yang diusulkan tidak berlaku, pertahankan nama yang ada. - Setelah langkah di atas selesai, urutkan semua pengujian yang terkait dengan op yang sedang diuji secara alfabetis berdasarkan nama fungsi.
- Terus tambahkan pengujian hingga ccov menampilkan cakupan >= 90% untuk operasi tersebut.
-
- Pastikan semua batasan yang terkait dengan pengujian inferensi bentuk ada dalam file ini, dengan mengikuti pedoman penamaan yang sama seperti yang disebutkan di atas.
- Pindahkan pengujian inferensi bentuk dari file ops_stablehlo.mlir ke dalam file ini.
Di spec.md:
- Tambahkan link ke
stablehlo/tests/interpret/<op_mnemonic>.mlir
ke bagian "Contoh" (misalnya Contoh Lainnya). - Pastikan spesifikasi hanya memiliki 1 contoh.
- Pastikan contoh spesifikasi mengikuti panduan pengujian.
- Pastikan pengujian contoh spesifikasi dapat ditafsirkan.
- Pastikan contoh spesifikasi sama dengan yang ada di ODS.
- Tambahkan link ke
Di status.md:
- Perbarui kolom "Penafsir" menjadi
yes
.
- Perbarui kolom "Penafsir" menjadi