-
Notifications
You must be signed in to change notification settings - Fork 50
fix: stabilize A5 validation samples #642
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 1 commit
d4494c8
f5946ae
749bab8
c28cb19
60a915e
9a1cb17
447c37b
fd633a2
3f7d8d8
7317936
a540344
8c93352
cc42815
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -231,6 +231,15 @@ else | |
| fi | ||
| fi | ||
|
|
||
| pto_isa_has_symbol() { | ||
| local symbol="$1" | ||
| [[ -n "${symbol}" ]] || return 1 | ||
| find "${PTO_ISA_ROOT}/include" "${PTO_ISA_ROOT}/tests" \ | ||
| -type f \( -name '*.h' -o -name '*.hpp' -o -name '*.cpp' -o -name '*.cc' \) \ | ||
| -print0 2>/dev/null \ | ||
| | xargs -0 grep -F -q "${symbol}" | ||
| } | ||
|
|
||
| status=0 | ||
| ok_count=0 | ||
| fail_count=0 | ||
|
|
@@ -267,6 +276,12 @@ while IFS= read -r -d '' cpp; do | |
| log "SKIP: ${testcase} (SKIP_CASES)" | ||
| continue | ||
| fi | ||
| if [[ "${testcase}" == "partarg" ]] && ! pto_isa_has_symbol "TPARTARGMAX("; then | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
The new skip guard only verifies Useful? React with 👍 / 👎. |
||
| skip_count=$((skip_count + 1)) | ||
| printf "%s\tSKIP\t%s\tpto-isa missing TPARTARGMAX/TPARTARGMIN\n" "${testcase}" "${STAGE}" >> "${RESULTS_TSV}" | ||
| log "SKIP: ${testcase} (pto-isa missing TPARTARG intrinsics)" | ||
| continue | ||
| fi | ||
| if [[ "${testcase}" == "gemvmx" ]]; then | ||
| soc_lc="$(printf '%s' "${SOC_VERSION:-}" | tr '[:upper:]' '[:lower:]')" | ||
| if [[ "$soc_lc" != *"a5"* && "$soc_lc" != *"950"* ]]; then | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The heuristic for identifying
mscatter_indices_inputusesreversed(init_ptrs), which selects the last integer input as the indices. This is inconsistent with thetscatterlogic (which selects the first integer input) and may be incorrect depending on the operand order of theMSCATTERoperation. Inmscatter.py,arg0appears to be the indices andarg1the data; if so, this heuristic will misidentifyarg1as the indices, potentially breaking thecompare_bin_at_indiceslogic later. Consider ifinit_ptrs(picking the first) would be more appropriate or if a more robust identification method is needed.