That sounds like a good idea. Let me see what I can do.
Kit
On 26/04/2025 23:19, Pierre Muller via fpc-devel wrote:
You can do that if you want,
maybe it would also be better to avoid all benchmark results writing
in this conformance configuration as there is not enough averaging,
and because
You can do that if you want,
maybe it would also be better to avoid all benchmark results writing
in this conformance configuration as there is not enough averaging,
and because in many cases the time interval is too short to get any value.
Pierre
Le 26/04/2025 à 23:50, J. Gareth Moreton via f
If it's okay, can I rename the define from "IN_TESTS" to "CONFORMANCE"
or similar? Technically, the code in "bcase" and "bfloatfunc" are still
tests, but by default are benchmarking tests evaluating performace
rather than conformance tests verifying that the compiler has produced
correct code.
"Just one small remark: why do you use '$8' instead of simply '8'?"
Mostly habit because the tests use "Iteration and 7" to select the
index, so I use $8 for the count to indicate that the bitwise nature of
it is important.
Kit
On 25/04/2025 21:31, Pierre Muller via fpc-devel wrote:
Hi,
Hi,
I add the IN_TESTS conditional code to avoid getting timeout related failures,
inside testsuite.
I didn't notice that more than one iteration was needed for bfloatfunc.pp,
because I tried it with ITERATIONS=1 and it seemed to finish without failure...
I have no objection to your patch
Hi everyone,
I just fixed an issue with tests/bench/bfloatfunc.pp over at
https://gitlab.com/freepascal.org/fpc/source/-/merge_requests/996
failing due to the reduced iteration count. I've increased it from 1 to
8 when in conformance mode (in IN_TESTS defined), since if it's less
than that,