[vector] unify load/store tests

[Lupusoru, Razvan A]

Wow! The patch looks great! Good job and thanks so much for doing this.

Everything looks good except one change - it seems that you removed check for ByteBuffer which only intrinsifies if ByteBuffer is native order. I wonder if you should keep that check in place since intrinsification leads to instructions that use native ordering while the input ByteBuffer may be marked as the other ordering. I believe the previous implementation was correct because it did not override the ByteBuffer ordering... Or did you have another plan for fixing the byte ordering issue?

--Razvan

-----Original Message-----
From: panama-dev [mailto:panama-dev-bounces at openjdk.java.net] On Behalf Of Paul Sandoz
Sent: Tuesday, May 22, 2018 11:31 AM
To: panama-dev at openjdk.java.net
Subject: [vector] unify load/store tests

Hi,

Here is an updated patch for unifying load/store. This is merged with the load/store tests.

  http://cr.openjdk.java.net/~psandoz/panama/unify-load-store/webrev/

- The relative buffer accessors are removed.

- The load/store intrinsics take a base + address for intrinsic access and a container + index for default access.

- Related methods were made abstract or final on the public primitive vectors.

- Redundant casts were removed from the implementation.

- Type variable names of intrinsic methods were changed for those that accept a Vector or Mask (alas we cannot represent this in the type system unless Vector and Mask share are common super type other than Object).

All tests pass locally for me in fast debug mode with -XX:+UseVectorApiIntrinsics -XX:-TieredCompilation.

Thanks,
Paul.