RFR: 8322758: Eliminate -Wparentheses warnings in C2 code [v2]

Kim Barrett kbarrett at openjdk.org
Tue Jan 2 22:27:01 UTC 2024


> 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.

Kim Barrett has updated the pull request incrementally with one additional commit since the last revision:

  update copyrights for 2024

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/17199/files
  - new: https://git.openjdk.org/jdk/pull/17199/files/8acc005e..abacbe0e

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=17199&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17199&range=00-01

  Stats: 9 lines in 9 files changed: 0 ins; 0 del; 9 mod
  Patch: https://git.openjdk.org/jdk/pull/17199.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/17199/head:pull/17199

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


More information about the hotspot-compiler-dev mailing list