RFR: 8347405: MergeStores with reverse bytes order value [v11]

kuaiwei duke at openjdk.org
Wed Feb 26 10:58:32 UTC 2025


On Wed, 26 Feb 2025 06:36:30 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

> To me the 3 nitpicks are optional, you don't have to apply them. Also I saw we already tested commit 16 / v10. I did launch testing again, just because there were a few weeks since the last time and sometimes bugs sneak via bad merges.
> 
> Approved. But it would be good to have a second reviewer give an approval too.
> 
> I'm excited to see what you do with this one 😊 [JDK-8345485](https://bugs.openjdk.org/browse/JDK-8345485) C2 MergeLoads: merge adjacent array/native memory loads into larger load

Thanks for your approval. I've fixed as comments.

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

PR Comment: https://git.openjdk.org/jdk/pull/23030#issuecomment-2684606802


More information about the hotspot-compiler-dev mailing list