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

Roberto Castañeda Lozano rcastanedalo at openjdk.org
Mon Mar 24 10:35:18 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?

Thanks for re-reviewing and the additional suggestions @eme64!

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

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


More information about the hotspot-compiler-dev mailing list