March 2021 - aarch64-port-dev

git: openjdk/aarch64-port: master: 8262185: G1: Prune collection set candidates early
by Thomas Schatzl 01 Mar '21

01 Mar '21

Changeset: 702ca622 Author: Thomas Schatzl <tschatzl(a)openjdk.org> Date: 2021-03-01 11:05:41 +0000 URL: https://git.openjdk.java.net/aarch64-port/commit/702ca622 8262185: G1: Prune collection set candidates early Reviewed-by: iwalulya, sjohanss, ayang ! src/hotspot/share/gc/g1/g1CollectionSetCandidates.cpp ! src/hotspot/share/gc/g1/g1CollectionSetCandidates.hpp ! src/hotspot/share/gc/g1/g1CollectionSetChooser.cpp ! src/hotspot/share/gc/g1/g1CollectionSetChooser.hpp ! src/hotspot/share/gc/g1/g1Policy.cpp ! src/hotspot/share/gc/g1/g1Policy.hpp

1 0

git: openjdk/aarch64-port: master: 8262195: Harden tests that use the HostsFileNameService (jdk.net.hosts.file property)
by Aleksei Efimov 01 Mar '21

01 Mar '21

Changeset: 8bc8542e Author: Conor Cleary <ccleary(a)openjdk.org> Committer: Aleksei Efimov <aefimov(a)openjdk.org> Date: 2021-03-01 10:18:12 +0000 URL: https://git.openjdk.java.net/aarch64-port/commit/8bc8542e 8262195: Harden tests that use the HostsFileNameService (jdk.net.hosts.file property) Reviewed-by: michaelm, aefimov, dfuchs, chegar - test/jdk/java/net/Inet4Address/TestToNumericFormatHosts ! test/jdk/java/net/Inet4Address/textToNumericFormat.java ! test/jdk/java/net/InetAddress/InternalNameServiceTest.java ! test/jdk/java/net/InetAddress/InternalNameServiceWithHostsFileTest.java ! test/jdk/java/net/InetAddress/InternalNameServiceWithNoHostsFileTest.java - test/jdk/java/net/InetAddress/TestHosts ! test/jdk/java/net/URLPermission/nstest/LookupTest.java ! test/jdk/sun/net/InetAddress/nameservice/simple/CacheTest.java - test/jdk/sun/net/InetAddress/nameservice/simple/CacheTestHosts ! test/jdk/sun/net/InetAddress/nameservice/simple/DefaultCaching.java - test/jdk/sun/net/InetAddress/nameservice/simple/DefaultCachingHosts

1 0

git: openjdk/aarch64-port: master: 8261914: IfNode::fold_compares_helper faces non-canonicalized bool when running JRuby JSON workload
by Roland Westrelin 01 Mar '21

01 Mar '21

Changeset: 20c93b3b Author: Roland Westrelin <roland(a)openjdk.org> Date: 2021-03-01 08:47:59 +0000 URL: https://git.openjdk.java.net/aarch64-port/commit/20c93b3b 8261914: IfNode::fold_compares_helper faces non-canonicalized bool when running JRuby JSON workload Reviewed-by: kvn, shade ! src/hotspot/share/opto/ifnode.cpp

1 0

git: openjdk/aarch64-port: master: 8261308: C2: assert(inner->is_valid_counted_loop(T_INT) && inner->is_strip_mined()) failed: OuterStripMinedLoop should have been removed
by Roland Westrelin 01 Mar '21

01 Mar '21

Changeset: ddd550ae Author: Roland Westrelin <roland(a)openjdk.org> Date: 2021-03-01 08:42:10 +0000 URL: https://git.openjdk.java.net/aarch64-port/commit/ddd550ae 8261308: C2: assert(inner->is_valid_counted_loop(T_INT) && inner->is_strip_mined()) failed: OuterStripMinedLoop should have been removed Reviewed-by: chagedorn, kvn ! src/hotspot/share/opto/loopTransform.cpp + test/hotspot/jtreg/compiler/loopopts/TestCountedLoopZeroIter.java

1 0

FW: RFR: 8257137: Revise smov and umov in aarch64 assembler [v4]
by Hao Sun 01 Mar '21

01 Mar '21

Can anyone help to review this patch? Thanks a lot. -----Original Message----- From: hotspot-compiler-dev <hotspot-compiler-dev-retn(a)openjdk.java.net> On Behalf Of Hao Sun Sent: Thursday, February 25, 2021 9:28 AM To: hotspot-compiler-dev(a)openjdk.java.net Subject: Re: RFR: 8257137: Revise smov and umov in aarch64 assembler [v4] > 1. Both smov and umov lack of checking the register type validity. > Register type must be 'B', 'H' or 'S' for smov [1]. > Register type can NOT be 'Q' for umov [2]. > Such checks are added. > > 2. smov and umov have different explanations on 'Q' field, i.e. bit 30 > of the insturction, but current assembler implementation mixed it up. > For umov, 'Q' field can only be set when register type 'D' is given > [2]. However, this field of smov must be set for register type 'S' > [1], that is, 'Q' field can be optional for register type 'B' or 'H'. > > Current implementation only took the umov scenario into account. As a > result, runtime error ILL_ILLOPN would occur if 'smov(Register, > FloatRegister, S, index)' is used. > > We put them into two separate functions and make 'Q' field always set > for smov. That means 'SMOVX' (64-bit register variant) is generated > for all cases since it's compatible with our current usages of 'SMOVW'. > Existing usages of smov have been double checked and this patch does > not affect them. > > 3. Smoke tests are also added. > > [1]. > https://developer.arm.com/docs/ddi0602/f/simd-and-floating-point-instr > uctions-alphabetic-order/smov-signed-move-vector-element-to-general-pu > rpose-register [2]. > https://developer.arm.com/docs/ddi0602/f/simd-and-floating-point-instr > uctions-alphabetic-order/umov-unsigned-move-vector-element-to-general- > purpose-register > > > Note that Jtreg tier1 and jdk::tier3 have been tested and all tests passed without new failures. Hao Sun has updated the pull request incrementally with one additional commit since the last revision: Code style: add spaces between operands add spaces between operands ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/1586/files - new: https://git.openjdk.java.net/jdk/pull/1586/files/ba69f500..b624a6b4 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1586&range=03 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1586&range=02-03 Stats: 75 lines in 2 files changed: 2 ins; 0 del; 73 mod Patch: https://git.openjdk.java.net/jdk/pull/1586.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1586/head:pull/1586 PR: https://git.openjdk.java.net/jdk/pull/1586 IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

1 0