Integrated: 8322758: Eliminate -Wparentheses warnings in C2 code

Kim Barrett kbarrett at openjdk.org
Tue Jan 2 22:36:24 UTC 2024


On Fri, 29 Dec 2023 02:01:08 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

> Please review this change to eliminate some -Wparentheses warnings.  In most
> cases, this involved simply adding a few parentheses to make some implicit
> operator precedence explicit.
> 
> In PhaseIdealLoop::rc_predicate, I also added a comment describing the test
> being performed, since it didn't seem obvious even with the additional
> parentheses.
> 
> Testing: mach5 tier1
> 
> Also ran mach5 tier1 with these changes in conjunction enabling -Wparentheses
> and other changes needed to make that work.

This pull request has now been integrated.

Changeset: 122bc777
Author:    Kim Barrett <kbarrett at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/122bc7770e1487cc754e17b9356217009bd6b13e
Stats:     27 lines in 9 files changed: 2 ins; 0 del; 25 mod

8322758: Eliminate -Wparentheses warnings in C2 code

Reviewed-by: aph, kvn

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

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


More information about the hotspot-compiler-dev mailing list