Hello Severin,
Even if using the macro, I still think you need to add a condition on
the compiler types where the switch can be reasonably expected to exist.
/Erik
On 2018-09-11 05:02, Severin Gehwolf wrote:
On Mon, 2018-09-10 at 09:29 -0700, Erik Joelsson wrote:
I see. I was not aware of that issue, but we clearly need to file a bug
for it and fix it. In this case I think it's fine to us the macro however.
OK. Update webrev, which now uses FLAGS_COMPILER_CHECK_ARGUMENTS.
http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8210416/webrev.04/
Micro-benchmark results from running [1] for x86_64 and ppc64le are
here (-O2 is sufficient it seems):
http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8210416/microbenchmarks_results/
More thoughts?
Thanks,
Severin
[1] https://github.com/gromero/strictmath/