RFR: 8315024: Vector API FP reduction tests should not test for exact equality [v2]

[Gergö Barany]

On Fri, 13 Oct 2023 10:25:55 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> @eme64 would you have time to take another look at the changes I have made to this PR?
>
> @gergo- I just looked at it again. It looks better.
> 
> Still, I have a concern about `cornerCaseValues`: Does the result not always degenerate to zero / NaN now?
> Every 17th value is a corner-case. And there, we mix in all of the special cases deterministically (zero, infty, min/max, NaN), depending on INT/FP type. But it seems that way we always have a NaN in the FP array - and therefore the reduction would always be NaN. Similar things happen with int-multiplication and zero.
> 
> I wonder if it would not be better to generate some data randomly, and throw in the special-cases with a very low probability. Maybe 50% that any show up, and then randomly pick one or more special case values. That way you can test the different special-cases separately. And their position could also be random.
> 
> BTW: is there any wiki about the template file format, and how to "compile" it to java? I might want to use it in the future myself :)

Thanks for the review @eme64 !

-------------

PR Comment: https://git.openjdk.org/jdk/pull/16024#issuecomment-1765793737