RFR: 8358641: C1 option -XX:+TimeEachLinearScan is broken

[Christian Hagedorn]

On Mon, 23 Jun 2025 09:43:28 GMT, Saranya Natarajan <snatarajan at openjdk.org> wrote:

> **Issue** 
> Using the command` java -Xcomp -XX:TieredStopAtLevel=1 -XX:+TimeEachLinearScan` results in an assert failure in line `assert(_cached_blocks.length() == ir()->linear_scan_order()->length()) failed: invalid cached block list`.
> 
> **Suggestion**
> Removal of flag as this is a very old issue 
> 
> **Fix**
> Removed the flag by removing relevant methods and code while ensuring the removal does not affect other flags.

That looks reasonable, thanks for cleaning it up!

src/hotspot/share/c1/c1_LinearScan.cpp line 6453:

> 6451:     case counter_other_inst:      return "misc. instructions";
> 6452: 
> 6453: 

Might have been added by a previous iteration. I suggest to remove it.
Suggestion:

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

Marked as reviewed by chagedorn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/25933#pullrequestreview-2993286243
PR Review Comment: https://git.openjdk.org/jdk/pull/25933#discussion_r2189710385