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