On Fri, 24 Apr 2026 07:10:52 GMT, Marc Chevalier <[email protected]> wrote:
>> I did a pass over the compiler code and cleaned up some code. This including: >> - Unused variables >> - Unused includes >> - Make methods `const/static` >> - Adding `override` >> - Fix code style >> - Rename variables that hide outer scope variables >> - Remove unused code >> - Remove line breaks not present in mainline >> >> More details in PR comments. >> >> I did some sanity testing up to tier4 + stress. >> >> Thanks, >> Christian >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > src/hotspot/share/ci/ciRefArrayKlass.hpp line 48: > >> 46: >> 47: ciKlass* exact_klass() override { >> 48: return this; > > Is it worth putting that on a single line, like the other methods above? Good point! Updated. ------------- PR Review Comment: https://git.openjdk.org/valhalla/pull/2364#discussion_r3137323710
