[foreign-memaccess] [Rev 01] RFR: Add MemorySegment::mismatch

[Chris Hegarty]

On Wed, 20 May 2020 14:26:24 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

>> Chris Hegarty has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Move implementation into vectorizedMismatchLarge, and address other review comments.
>
> test/jdk/java/foreign/TestMismatch.java line 99:
> 
>> 98:         }
>> 99:     }
>> 100:
> 
> How important is it that these tests operate on slices? Looking at the test code, it could have worked equally well if
> the input parameters were just two sizes, and then you did an explicit allocation (or maybe also receive a segment
> factory from the provider, so that you can test different segment kinds).

Originally I had a version of the test that did compare specific segments, but it didn't scale well to different sizes
and kinds ( we need to test both above and below the 8 byte threshold ). I removed a number of slice sizes, which
greatly reduces the combinations. This may be enough, or I can certainly revisit the test's structure.

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

PR: https://git.openjdk.java.net/panama-foreign/pull/180