Integrated: 8323115: x86-32: Incorrect predicates for cmov instruct transforms with UseSSE

Kim Barrett kbarrett at openjdk.org
Tue Jan 9 22:36:52 UTC 2024


On Mon, 8 Jan 2024 07:55:21 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

> Please review this change that fixes generation of CMOV by C2 as controlled by
> UseSSE.  The predicates controlling that generation were using implicit
> operator precedence that didn't have the expected grouping.  Fixed by adding
> parentheses to make the desired grouping explicit.
> 
> Testing: Ran GHA with -Wparentheses enabled along with this and other changes
> needed to make that work.

This pull request has now been integrated.

Changeset: 28d8149c
Author:    Kim Barrett <kbarrett at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/28d8149c693a9470bbde4b1a27c4b9be6c5f365c
Stats:     13 lines in 1 file changed: 0 ins; 0 del; 13 mod

8323115: x86-32: Incorrect predicates for cmov instruct transforms with UseSSE

Reviewed-by: shade, thartmann

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

PR: https://git.openjdk.org/jdk/pull/17296


More information about the hotspot-compiler-dev mailing list