RFR: 8351468: C2: array fill optimization assigns wrong type to intrinsic call

Roberto Castañeda Lozano rcastanedalo at openjdk.org
Mon Mar 24 08:30:12 UTC 2025


On Fri, 21 Mar 2025 15:15:00 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>>> What would be a better name though?
>> 
>> @merykitty had the suggestions `MemNode::value_type()` or `MemNode::value_basic_type()` (see comment [above](https://github.com/openjdk/jdk/pull/24005#issuecomment-2724445592)), I like both better than the current name.
>
>> > What would be a better name though?
>> 
>> @merykitty had the suggestions `MemNode::value_type()` or `MemNode::value_basic_type()` (see comment [above](https://github.com/openjdk/jdk/pull/24005#issuecomment-2724445592)), I like both better than the current name.
> 
> @merykitty @robcasloz `MemNode::value_basic_type()` sounds like the most descriptive and accurate. Great! @robcasloz , will you file an RFE for that?

Hi @eme64, I implemented your test suggestions (commit b59d2eb2), please re-review.

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

PR Comment: https://git.openjdk.org/jdk/pull/24005#issuecomment-2747277559


More information about the hotspot-compiler-dev mailing list