RFR: 8332856: C2: Add new transform for bool eq/ne (cmp (and (urshift X const1) const2) 0) [v3]
Tobias Hotz
duke at openjdk.org
Mon Jul 22 19:31:10 UTC 2024
> This PR adds a new ideal optimization for the following pattern:
>
> public boolean testFunc(int a) {
> int mask = 0b101;
> int shift = 12;
> return ((a >> shift) & mask) == 0;
> }
>
> Where the mask and shift are constant values and a is a variable. For this optimization to work, the right shift has to be idealized to a unsinged right shift earlier in the pipeline, which here: https://github.com/openjdk/jdk/blob/b92bd671835c37cff58e2cdcecd0fe4277557d7f/src/hotspot/share/opto/mulnode.cpp#L731
> If the shift is already an unsiged bit shift, it works as well.
> On AMD64 CPUs, this means that this whole line computation can be reduced to a simple `test` instruction.
Tobias Hotz has updated the pull request incrementally with one additional commit since the last revision:
Adress review comments
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/19310/files
- new: https://git.openjdk.org/jdk/pull/19310/files/3491ceb3..426de3b8
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=19310&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=19310&range=01-02
Stats: 3 lines in 2 files changed: 0 ins; 0 del; 3 mod
Patch: https://git.openjdk.org/jdk/pull/19310.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/19310/head:pull/19310
PR: https://git.openjdk.org/jdk/pull/19310
More information about the hotspot-compiler-dev
mailing list