Preparing for enabling -docc-tune=tenstorrent on libOpenFOAM #34
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Member
ramonwirsch
commented
Dec 12, 2025
ramonwirsch
commented
- renamed confusingly named test source to match its _test binary
- upgraded tests/Amul to use the refAmul to verify results
- disable docc on test sources if that is the compiler used
- set other flags needed
* renamed confusingly named test source to match its _test binary * upgraded tests/Amul to use the refAmul to verify results * disable docc on test sources if that is the compiler used * set other flags needed
Daisytuner Report - tt_kernel_performance_benchs (zinnia)@@ Benchmarks @@
=====================================================================================
Benchmark Time ΔTime Thr Energy ΔEnergy
=====================================================================================
# denseMatrix_Amul 3.88 s +0.45% N/A N/A N/A
# denseMatrix_Amul_test 3.34 s +0.21% N/A N/A N/A
# ellpackMatrix_Amul_fpu 3.17 s -1.39% N/A N/A N/A
# ellpackMatrix_Amul_no_hw3.17 s -1.40% N/A N/A N/A |
Daisytuner Report - tt_kernel_performance_benchs (zinnia-ci)@@ Benchmarks @@
=====================================================================================
Benchmark Time ΔTime Thr Energy ΔEnergy
=====================================================================================
# denseMatrix_Amul 5.68 s +0.02% N/A N/A N/A
# denseMatrix_Amul_test 5.18 s -0.00% N/A N/A N/A
# ellpackMatrix_Amul_fpu 4.98 s +0.01% N/A N/A N/A
# ellpackMatrix_Amul_no_hw4.98 s -0.00% N/A N/A N/A |
Member
Author
|
Docc version with needed cmd options not yet released |
+ saving range of cols referenced in each tile to make Tensix cores no longer have to stream ALL of the vector, but only the relevant parts ! closed range of vectors across the batched tiles is loaded per core (considering alignment to 1024 vectors and 128 vector pages) ~ fixed lack of synchronization in soft-float impl, cb_get_tile semaphore was not guaranteed clean ! currently, T0 cannot collect next batch
ramonwirsch
force-pushed
the
docc-tune-tt
branch
from
b212462 to
5d7665f
Compare
…am / lduMat specific meta data structs and is more reusable
ramonwirsch
force-pushed
the
docc-tune-tt
branch
from
2531055 to
461c5a4
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.