RFR: 8327963: [Umbrella] Incorrect result of C2 compiled code since JDK-8237581 [v2]

Wed Apr 23 12:38:59 UTC 2025

On Wed, 23 Apr 2025 12:26:43 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> src/hotspot/share/opto/multnode.hpp line 135:
>> 
>>> 133:   }
>>> 134:   virtual int Opcode() const;
>>> 135: };
>> 
>> Would it make sense to have these overridden? Just so you can print the `_adr_type` :)
>> 
>> #ifndef PRODUCT
>>   virtual void dump_spec(outputStream *st) const;
>>   virtual void dump_compact_spec(outputStream *st) const;
>> #endif
>
> Ah, or does it already get printed from the `adr_type()` i.e. the virtual method?

Hmm, no I don't think so... right?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24570#discussion_r2055938496