RFR: 8241502: C2: Migrate x86_64.ad to MacroAssembler [v8]

John Tortugo github.com+2249648+johntortugo at openjdk.java.net
Thu Apr 29 20:41:59 UTC 2021


On Thu, 22 Apr 2021 23:50:29 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> John Tortugo has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix cast of constant
>
> I got a lot of failures on Windows only. Almost all test/hotspot/jtreg/serviceability/sa/ tests failed and others.
> Examples:
> 
> serviceability/sa/TestSysProps.java
> 
> #  Internal Error (t:\\workspace\\open\\src\\hotspot\\share\\runtime\\stackValue.cpp:139), pid=7572, tid=26052
> #  assert(oopDesc::is_oop_or_null(val, false)) failed: bad oop found
> 
> compiler/c2/Test6800154.java
> java.lang.InternalError: 9223372036854775807 / -9223372036854775808 failed: 65547 != 0
> 	at compiler.c2.Test6800154.run(Test6800154.java:111)
> 	at compiler.c2.Test6800154.main(Test6800154.java:97)

@vnkozlov - thank you so much for running the tests! The cause of the problems you reported were the last changes I made to the div/mod instructions. I fixed the code and ran all tests again on Linux, macOS, and Windows and they are looking good (jdk tier1, 2, 3, and hotspot_all.

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

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


More information about the hotspot-compiler-dev mailing list