RFR: 8304034: Remove redundant and meaningless comments in opto [v5]

Yi Yang yyang at openjdk.org
Tue Mar 14 05:34:20 UTC 2023


> Please help review this trivial change to remove redundant and meaningless comments in `hotspot/share/opto` directory.
> 
> They are either
> 1. Repeat the function name that the function they comment for.
> 2. Makes no sense, e.g. `//----Idealize----` 
> 
> And I think original CC-style code (`if( test )`,`call( arg )`) can be formatted in one go, instead of formatting the near code when someone touches them. But this may form a big patch, and it confuses code blame, so I left this work until we reach a consensus.
> 
> Thanks!

Yi Yang has updated the pull request incrementally with one additional commit since the last revision:

  restore mistakenly removed lines

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/12995/files
  - new: https://git.openjdk.org/jdk/pull/12995/files/b3fc99e3..92963630

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=12995&range=04
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=12995&range=03-04

  Stats: 37 lines in 1 file changed: 34 ins; 3 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/12995.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/12995/head:pull/12995

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


More information about the hotspot-compiler-dev mailing list