RFR: 8343941: IGV: dump graph at different register allocation steps [v2]
Damon Fenacci
dfenacci at openjdk.org
Tue Nov 12 13:45:25 UTC 2024
On Tue, 12 Nov 2024 11:55:09 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:
>> This changeset dumps C2's low-level intermediate representation at the following intermediate register allocation points:
>> - Initial liveness: after initial liveness information is computed.
>> - Aggressive coalescing: after aggressively coalescing live ranges and destructing SSA.
>> - Initial spilling: after initial round of spilling derived from physical interference graph construction.
>> - Conservative coalescing: after each round of conservative (colorability-preserving) coalescing (if `OptoCoalesce` is enabled).
>> - Iterative spilling: after each round of spilling.
>> - After iterative spilling: after the main register allocation loop.
>> - Post-allocation copy removal: after peephole copy removal.
>> - Merge multiple definitions: after local merging of equivalent nodes related by the same live range.
>> - Fix up spills: convert load-store spills into memory operand accesses ("CISC spilling") if allowed by the target platform and `UseCISCSpill` is enabled.
>>
>> The new dumps have already proved to be useful in the investigation of [JDK-8331295](https://bugs.openjdk.org/browse/JDK-8331295).
>>
>> #### Testing
>>
>> - tier1-3 (windows-x64, linux-x64, linux-aarch64, macosx-x64, and macosx-aarch64).
>> - Tested automatically that dumping, scheduling, and viewing hundreds of the new graphs does not trigger any failure on HotSpot or IGV (by instrumenting IGV to schedule and view graphs eagerly and running `java -Xbatch -XX:PrintIdealGraphLevel=4`).
>
> Roberto Castañeda Lozano has updated the pull request incrementally with one additional commit since the last revision:
>
> Split FIXUP_SPILLS
Very very cool!
@robcasloz do you think it could make sense to add a few IR tests just to make sure that the new steps are actually dumped?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/22017#issuecomment-2470555444
More information about the hotspot-compiler-dev
mailing list