RFR: 8348678: [PPC64] C2: unaligned vector load/store is ok [v2]

Martin Doerr mdoerr at openjdk.org
Wed Jan 29 17:41:46 UTC 2025


On Wed, 29 Jan 2025 15:51:03 GMT, Richard Reingruber <rrich at openjdk.org> wrote:

>> This pr changes `Matcher::misaligned_vectors_ok`  to return `true` on PPC64 for better vectorization during `SuperWord`.
>> IR checking of the corresponding test `TestCastX2NotProcessedIGVN.java` is also enabled.
>> 
>> Tested with `TestCastX2NotProcessedIGVN.java`
>> 
>> The change passed our CI testing:
>> Tier 1-4 of hotspot and jdk. All of langtools and jaxp. Renaissance Suite and SAP specific tests.
>> Testing was done on the main platforms and also on Linux/PPC64le and AIX.
>
> Richard Reingruber has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Use -AlignVector by default for POWER8 and later

The displacement should be handled by the `memoryAlg4` operands. (Should be checked.) It may make sense to switch off `AlignVector` even without `SuperwordUseVSX` (on any Power Architecture).

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

PR Comment: https://git.openjdk.org/jdk/pull/23318#issuecomment-2622416594


More information about the hotspot-compiler-dev mailing list