RFR: 8251994: VM crashed running TestComplexAddrExpr.java test with -XX:UseAVX=X

[Aleksey Shipilev]

On Mon, 26 Oct 2020 15:51:36 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

> 'fragile' is in naming sense to use `DO_VECTOR_LOOP_EXPERIMENTAL`, for example?

This makes most sense, I believe.

> Or you prefer to have the code be guarded by `if(_do_vector_loop_experimental)` runtime check instead of `#if DO_VECTOR_LOOP_EXPERIMENTAL` macro check?

Yeah, I am not a big fan of doing macros when runtime checks carry the same weight. I understand you want to micro-optimize these paths with macros, but I think the runtime checks work without much penalty here? I would defer to compiler reviewers to say which one is better.

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

PR: https://git.openjdk.java.net/jdk/pull/859