RFR: 8322589: Add Ideal transformation: (~a) & (~b) => ~(a | b) [v5]

[Emanuel Peter]

On Mon, 8 Jan 2024 07:02:50 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Zhiqiang Zang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   update the copyright dates.
>
> src/hotspot/share/opto/addnode.hpp line 84:
> 
>> 82:   // Utility function to check if the given node is a NOT operation,
>> 83:   // i.e., n == m ^ (-1).
>> 84:   static bool is_not(PhaseGVN* phase, Node* n, BasicType bt);
> 
> Could these be made non-static?

Hmm, I agree with this idea. `n->is_not(...)` would really be nicer. You'd probably have to move the two methods to `node.hpp`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16333#discussion_r1444251176