65831 Commits

Author SHA1 Message Date
Faye Gao
f87c7ddc65 8272968: AArch64: Remove redundant matching rules for commutative ops
Match rules for commutative operations mnegI/mnegL/smnegL might
become redundant after function matchrule_clone_and_swap(),
and hence can be reduced.

In adlc part, while parsing the contents of an instruction
definition, function instr_parse always do the check
for commutative operations with subtree operands, create
clones and swap operands by function matchrule_clone_and_swap.
It means that another operand-swapped and partially
symmetrical match rule should be generated automatically for
these commutative operations.

The pattern to construct mnegI, mnegL or smnegL consists of
a subtraction with zero and then a multiplication. In function
count_commutative_op, both mulI and mulL are recognized as
commutative opcodes. Therefore, we need only one match rule
to specify that a multipilication consists of a number and
a subtraction with zero for these three instructions and the
extra one can be deleted.

Take mnegL as an example.

Without my patch, four match rules will be created finally for
instruction selection.

Two of them are created by ad files:

Match Rule 1:
dst = MulL (SubL zero src1) src2
===>
dst = mnegl src1 src2

Match Rule 2:
dst = MulL src1 (SubL zero src2)
===>
dst = mnegl src1 src2

The other two are automatically generated by function
matchrule_clone_and_swap based on the two rules above:

Match Rule 3 (generated by match rule 1):
dst = MulL src2 (SubL zero src1)
===>
dst = mnegl src1 src2

Match Rule 4 (generated by match rule 2):
dst = MulL (SubL zero src2) src1
===>
dst = mnegl src1 src2

As mnegl is commutative, Rule 3 is equivalent to
Rule 2, and Rule 1 is equivalent to Rule 4. Also, if we only
keep the original Match Rule 1, as showed above, Rule 3 will
be generated automatically later. In this way, Rule 2 and Rule 4
are redundant and hence Rule 2 can be eliminated.

With my patch, Rule 2 is removed and Rule 4 won't be generated as well.
Only Rule 1 and 3 are kept in the final rule chain. In my local release
build, as redundant code got removed, the size of libjvm.so decreased
from 23.30 MB to 23.27 MB, with a reduction of 33.11 KB(around 0.14%).
2021-08-31 05:57:11 +00:00
Yumin Qi
9732fbe428 8273153: Consolidate file_exists into os:file_exists
Reviewed-by: iklam, ccheung
2021-08-31 00:24:05 +00:00
Vicente Romero
0609421d4b 8272347: ObjectMethods::bootstrap should specify NPE if any argument except lookup is null
Reviewed-by: mchung, chegar
2021-08-30 21:16:46 +00:00
Naoto Sato
7fc8540907 8260265: UTF-8 by Default
Reviewed-by: alanb, rriggs
2021-08-30 21:13:59 +00:00
Ioi Lam
32048536e9 8272343: Remove MetaspaceClosure::FLAG_MASK
Reviewed-by: ccheung, minqi
2021-08-30 21:06:26 +00:00
Ian Graves
fecefb8541 8271302: Regex Test Refresh
Reviewed-by: bchristi, smarks
2021-08-30 17:37:29 +00:00
Fernando Guallini
f18c0fac11 8271560: sun/security/ssl/DHKeyExchange/LegacyDHEKeyExchange.java still fails due to "An established connection was aborted by the software in your host machine"
Reviewed-by: xuelei, rhalade
2021-08-30 17:28:49 +00:00
Sandhya Viswanathan
5aaa20f898 8272861: Add a micro benchmark for vector api
Reviewed-by: psandoz
2021-08-30 16:48:28 +00:00
Thomas Schatzl
7a01ba6528 8272093: Extract evacuation failure injection from G1CollectedHeap
Reviewed-by: ayang, sjohanss
2021-08-30 15:54:47 +00:00
Coleen Phillimore
98b9d98032 8272797: Mutex with rank safepoint_check_never imply allow_vm_block
Reviewed-by: dholmes, pchilanomate
2021-08-30 14:47:24 +00:00
Stefan Johansson
f11e099a14 8272651: G1 heap region info print order changed by JDK-8269914
Reviewed-by: tschatzl, iwalulya, ayang
2021-08-30 14:37:03 +00:00
Magnus Ihse Bursie
fbffa54efe 8270438: "Cores to use" output in configure is misleading
Reviewed-by: erikj
2021-08-30 13:41:54 +00:00
Andrey Turbanov
5185dbde67 8273098: Unnecessary Vector usage in java.naming
Reviewed-by: aefimov, dfuchs
2021-08-30 13:12:42 +00:00
Lutz Schmidt
276b07b36a 8271490: [ppc] [s390]: Crash in JavaThread::pd_get_top_frame_for_profiling
Reviewed-by: stuefe, mbaesken
2021-08-30 12:31:08 +00:00
Thomas Schatzl
bb7aa1c6a4 8272161: Make evacuation failure data structures local to collection
Reviewed-by: iwalulya, sjohanss
2021-08-30 12:08:12 +00:00
Magnus Ihse Bursie
9ede41bf89 8229031: Exporting CLASSPATH from shell can result in build failures
Reviewed-by: erikj
2021-08-30 11:37:47 +00:00
Michael McMahon
16e83058ca 8273059: Redundant Math.min call in Http2ClientImpl#getConnectionWindowSize
Reviewed-by: dfuchs
2021-08-30 08:57:17 +00:00
Aleksey Shipilev
f55d5ab517 8272838: Move CriticalJNI tests out of tier1
Reviewed-by: dholmes
2021-08-30 06:41:40 +00:00
Guoxiong Li
a9188f237e 8268894: forged ASTs can provoke an AIOOBE at com.sun.tools.javac.jvm.ClassWriter::writePosition
Reviewed-by: vromero
2021-08-29 07:26:08 +00:00
Bradford Wetmore
1fb798d320 8272915: (doc) package-info typo in extLink
Reviewed-by: xuelei
2021-08-27 23:01:51 +00:00
Raffaello Giulietti
51167846cb 8273091: Doc of [Strict]Math.floorDiv(long,int) erroneously documents int in @return tag
Reviewed-by: darcy, bpb
2021-08-27 22:48:52 +00:00
Jim Laskey
e66c8afb59 8272866: java.util.random package summary contains incorrect mixing function in table
Reviewed-by: rriggs
2021-08-27 18:45:52 +00:00
Brian Burkhalter
d1aeca117c 8272541: Incorrect overflow test in Toom-Cook branch of BigInteger multiplication
Reviewed-by: darcy
2021-08-27 16:09:28 +00:00
Brian Burkhalter
dfeb4132e4 8272964: java/nio/file/Files/InterruptCopy.java fails with java.lang.RuntimeException: Copy was not interrupted
Reviewed-by: dfuchs
2021-08-27 16:08:29 +00:00
Magnus Ihse Bursie
a033aa5a3d 8273072: Avoid using += in configure
Reviewed-by: dholmes, jiefu
2021-08-27 13:53:33 +00:00
Coleen Phillimore
b92214a8d0 8272480: Remove Mutex::access rank
Reviewed-by: dholmes, eosterlund
2021-08-27 13:51:39 +00:00
Magnus Ihse Bursie
596b075591 8258465: Headless build fails due to missing X11 headers on linux
Reviewed-by: shade
2021-08-27 13:13:36 +00:00
Kim Barrett
a49a0c5866 8273062: Generation::refs_discovery_is_xxx functions are unused
Reviewed-by: ayang
2021-08-27 10:32:46 +00:00
Aleksey Shipilev
ebd62bc043 8272846: Move some runtime/Metaspace/elastic/ tests out of tier1
Reviewed-by: mseledtsov, stuefe
2021-08-27 09:35:26 +00:00
Lin Zang
46684a4efa 8262386: resourcehogs/serviceability/sa/TestHeapDumpForLargeArray.java timed out
Reviewed-by: cjplummer, sspitsyn
2021-08-27 09:02:23 +00:00
Tobias Hartmann
c925c7f3e8 8273020: LibraryCallKit::sharpen_unsafe_type does not handle narrow oop array
Reviewed-by: chagedorn
2021-08-27 05:38:24 +00:00
Bradford Wetmore
76baace2f0 8273045: Fix misc javadoc bugs in the java.security and javax.net.ssl code
Reviewed-by: xuelei
2021-08-27 05:11:25 +00:00
Yasumasa Suenaga
b16a04ebf7 8271186: Add UL option to replace newline char
Reviewed-by: iklam, dholmes
2021-08-27 05:06:52 +00:00
Andrey Turbanov
d732c3091f 8272863: Replace usages of Collections.sort with List.sort call in public java modules
Reviewed-by: serb, dfuchs, naoto
2021-08-26 20:47:02 +00:00
Naoto Sato
fe7d70886c 8272473: Parsing epoch seconds at a DST transition with a non-UTC parser is wrong
Reviewed-by: joehw, rriggs, iris, lancea, scolebourne
2021-08-26 16:21:23 +00:00
Ivan Walulya
845e1cea8d 8272983: G1 Add marking details to eager reclaim logging
Reviewed-by: tschatzl, ayang
2021-08-26 13:41:46 +00:00
Alexey Ushakov
c420530549 8272481: [macos] javax/swing/JFrame/NSTexturedJFrame/NSTexturedJFrame.java fails
Reviewed-by: serb, jdv
2021-08-26 12:56:03 +00:00
Prasanta Sadhukhan
e43a907f20 8271315: Redo: Nimbus JTree renderer properties persist across L&F changes
Reviewed-by: aivanov
2021-08-26 12:04:10 +00:00
Albert Mingkun Yang
11c9fd8298 8272975: ParallelGC: add documentation to heap memory layout
Co-authored-by: Thomas Schatzl <tschatzl@openjdk.org>
Reviewed-by: tschatzl, kbarrett
2021-08-26 10:06:38 +00:00
Magnus Ihse Bursie
b94fd32f08 8272859: Javadoc external links should only have feature version number in URL
Reviewed-by: iris
2021-08-26 10:00:21 +00:00
Tobias Hartmann
9166ba37b6 8272973: Incorrect compile command used by TestIllegalArrayCopyBeforeInfiniteLoop
Reviewed-by: chagedorn, jiefu, neliasso
2021-08-26 08:34:46 +00:00
Alexey Ushakov
49b2789a49 8262751: RenderPipelineState assertion error in J2DDemo
Reviewed-by: jdv
2021-08-26 07:28:26 +00:00
Aleksey Shipilev
a3308af060 8272836: Limit run time for java/lang/invoke/LFCaching tests
Reviewed-by: redestad, iignatyev
2021-08-26 06:51:19 +00:00
David Holmes
c4c76e2f34 8272811: Document the effects of building with _GNU_SOURCE in os_posix.hpp
Reviewed-by: iklam
2021-08-26 02:24:36 +00:00
Igor Veresov
673ce7efa5 8272873: C2: Inlining should not depend on absolute call site counts
Reviewed-by: kvn, vlivanov, dlong
2021-08-26 00:17:15 +00:00
Zhengyu Gu
7212561dd1 8267188: gc/stringdedup/TestStringDeduplicationInterned.java fails with Shenandoah
Reviewed-by: rkennke, shade
2021-08-25 20:16:25 +00:00
Ivan Walulya
e36cbd8e05 8242847: G1 should not clear mark bitmaps with no marks
Reviewed-by: tschatzl, sjohanss
2021-08-25 14:33:34 +00:00
Coleen Phillimore
2ef6871118 8272447: Remove 'native' ranked Mutex
Reviewed-by: stuefe, pchilanomate
jdk-18+12
2021-08-25 13:25:34 +00:00
Ivan Walulya
63e062fb78 8236176: Parallel GC SplitInfo comment should be updated for shadow regions
Reviewed-by: tschatzl, ayang
2021-08-25 10:18:54 +00:00
Aleksey Shipilev
c5a271259d 8272850: Drop zapping values in the Zap* option descriptions
Reviewed-by: stefank, kbarrett
2021-08-25 08:05:47 +00:00