[PATCH] D83692: Add first microbenchmarks for matrix types extensions.
Sjoerd Meijer via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 21 02:22:27 PDT 2020
SjoerdMeijer added a comment.
Looks decent as an initial commit to me. Two high level question:
- I haven't looked at these MicroBenchmarks yets in the test-suite, but in general it would be convenient if a benchmarks also does a correctness check. Do you think there would be any value in doing that here? If so, would that easy to add?
- In benchmarking, stable numbers are convenient. Since the input is randomly generated, I was wondering if there could be timing differences depending on different inputs? But I guess not here?
Repository:
rT test-suite
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D83692/new/
https://reviews.llvm.org/D83692
More information about the llvm-commits
mailing list