Commit 23563fb
fix(tensilelite): Accept .dat.zlib in --use-cache existence check (#8892)
## Motivation
PR #8294 changed `writeMsgPack()` to compress library catalogs with
zlib, producing `.dat.zlib` files instead of `.dat`. The C++ runtime's
`fileToMsgObject()` was updated to probe for `.zlib` automatically, but
the Python `--use-cache` existence check in `BenchmarkProblems.py` was
missed — it still checks only for `.dat`, causing all `--use-cache`
benchmark tests to FATAL when `LibraryFormat=msgpack`.
## Technical Details
Widen the `os.path.isfile()` existence check to accept both `.dat` and
`.dat.zlib` on disk. The path passed to the C++ client stays as `.dat`,
matching the C++ probe convention from PR #8294 where
`fileToMsgObject()` tries `filename + ".zlib"` before the base file.
## Test Plan
- [x] Emulation CI: common tests pass with `LibraryFormat=msgpack`
- [x] No change to `tensileLibraryFile()` or `writeClientConfigIni` —
C++ client probe convention preserved
- [x] Jenkins `tox -e py3` (yaml format, unaffected path)
## Submission Checklist
- [x] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.
JIRA ID: AIHPBLAS-3508
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>1 parent 9cd69f1 commit 23563fb
2 files changed
Lines changed: 20 additions & 1 deletion
File tree
- projects/hipblaslt/tensilelite/Tensile
- Tests/unit
Lines changed: 3 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
758 | 758 | | |
759 | 759 | | |
760 | 760 | | |
761 | | - | |
| 761 | + | |
| 762 | + | |
| 763 | + | |
762 | 764 | | |
763 | 765 | | |
764 | 766 | | |
| |||
Lines changed: 17 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
83 | 83 | | |
84 | 84 | | |
85 | 85 | | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
86 | 103 | | |
87 | 104 | | |
88 | 105 | | |
| |||
0 commit comments