RFR: 8282555: Missing memory edge when spilling MoveF2I, MoveD2L etc [v3]

Emanuel Peter duke at openjdk.java.net
Wed May 4 05:17:02 UTC 2022


On Tue, 3 May 2022 13:09:50 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

>> Emanuel Peter has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Adapted comments again
>>  - compacted comment, added test without -Djdk.test.lib.random.seed=42
>
> src/hotspot/share/opto/chaitin.cpp line 1736:
> 
>> 1734:             // src to cisc, else we might schedule cisc before src, loading from a
>> 1735:             // spill location before storing the spill.
>> 1736:             cisc->add_prec(src);
> 
> An assertion for null memory operand can be added before adding presidence edge.

@jatin-bhateja Thanks, I added it :)

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

PR: https://git.openjdk.java.net/jdk/pull/7889


More information about the hotspot-compiler-dev mailing list