במסמך הזה אנחנו מסכם את ההנחיות לבדיקת השינויים במפרט. בשלב זה, השינויים האלה בדרך כלל כוללים בדיקה של כמה דברים במספר מקורות, ולכן המסמך הזה מסכם את כולם כדי לפשט את הבדיקות:
- בודקים שבעמודה "Specification" ב-status.md כתוב "yes". אם מוסיפים פעולה חדשה, מוסיפים שורה.
- בודקים אם שם הקטע תואם לכינוי של המבצע ב-ODS.
- בודקים אם הקטע 'סמנטיקה' תואם ל-Operation Semantics של XLA.
- בודקים אם הקטעים Inserts ו-Outputs:
- מציינים את אותם פריטים כמו ה-ODS.
- מציינים את אותם פריטים כמו HloInstruction::CreateFromProto.
- מסודרים בדיוק כמו ב-ODS.
- אם יש חוסר התאמה, בודקים שיש כרטיסים תואמים.
- בודקים אם הקטע "מגבלות":
- תואם ל-shape_inference.cc של XLA.
- תואם ל-hlo_verifier.cc של XLA.
- תואם ל-ODS.
- תואם ל-StablehloOps.cpp.
- אם יש חוסר התאמה, בודקים שיש כרטיסים תואמים. מקשרים את כל הכרטיסים האלה במפרט, במיקומים ספציפיים ככל האפשר (למשל, אם כרטיס עוסק באילוץ שלא יושם, צריך לקשר את הכרטיס ישירות באילוץ הזה).
- אם החלקים התואמים ב-ODS וב-StablehloOps.cpp תואמים למפרט, בודקים שבעמודות "Authentication" ו-"Type Inference" של status.md כתוב "yes".
- בודקים אם הקטע 'דוגמאות':
- יש רק דוגמה אחת. (בעתיד, נקשר לדוגמאות נוספות מחבילת הבדיקות של המתורגמן StableHLO).
- עושה שימוש בתחביר MLIR תקין על ידי הרצת
stablehlo-opt
על דוגמאות לקוד. - נעשה שימוש בתחביר MLIR גנרי על ידי הרצת
stablehlo-opt -mlir-print-op-generic
(אנחנו מקפידים על תחביר גנרי במפרט כדי להימנע מהצורך לשנות את המפרט בשינויים בהדפסה יפה).
- בודקים שה-
description
ב-ODS של ההפעלה:- כולל את המשפט הראשון של המפרט.
- לאחר מכן יש קישורים לקטע המתאים במפרט.
- לאחר מכן נשתמש באותה דוגמה כמו במפרט, אבל באמצעות תחביר יפה שיכול
להתקבל על ידי הרצת
stablehlo-opt
.
- בודקים שהקבצים שקשורים להטמעה של אילוצים לגבי אימות
והסקת מסקנות תואמים להנחיות שמפורטות בהמשך:
- פועלים לפי ההנחיות מס' 1 ל-StablehloOps.td.
- פועלים לפי ההנחיות מס' 2 ל-TypeInference.cpp ול-StablehloOps.cpp.
- פועלים לפי ההנחיות מס' 5 ל-ops_stablehlo.mlir.
- פועלים לפי ההנחיות מס' 6 לגבי infer_stablehlo.mlir.
- מעריכים את ההפעלה מבחינת השפעות לוואי וספקולציות.
- אם לפעולה אין תופעות לוואי ותמיד אפשר להתבסס על ספקולציות, צריך לתת לה את
ה-trait
Pure
. זהו מצב נדיר כי רוב הפעולות מאפשרות ליצור צורות דינמיות, דבר שעלול להוביל לחוסר התאמה בצורות בזמן הריצה, כלומר התנהגות לא מוגדרת. לחלק מהפעולות עשויה להיות התנהגות לא מוגדרת גם במצבים אחרים. לרוב המכריע של הפעולות אין תופעות לוואי (הן צריכות לכלול את המאפייןNoMemoryEffect
). - רוב הפעולות נכללות באחת ממאפייני ה-
HLO_SpeculatableIf*
. אם הפעולה לא מתאימה לאף אחת מהאפשרויות האלה, צריך לספק לה את המאפייןConditionallySpeculatable
ולהטמיע את שיטות הממשק. מוסיפים בדיקות ל-stablehlo/tests/ops_speculatability.mlir
כדי לכסות את לוגיקת הספקולציות.
- אם לפעולה אין תופעות לוואי ותמיד אפשר להתבסס על ספקולציות, צריך לתת לה את
ה-trait