On Mon, 10 Jun 2024 23:10:05 GMT, Shaojin Wen wrote:
>> @wenshao
>>> I think the performance of the Unsafe branch may be the best data for the
>>> C2 optimizer. @eme64 can help me see if C2 can do it?
>>
>> Have you tried to see if the optimization actually was done/taken? You can
>> use the
On Sun, 9 Jun 2024 19:35:04 GMT, Justin Lu wrote:
>> Damon Nguyen has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Remove double quotes
>
> src/jdk.jconsole/share/classes/sun/tools/jconsole/resources/messages_de.properties
> line 285:
>
On Mon, 10 Jun 2024 20:34:10 GMT, Damon Nguyen wrote:
>> src/jdk.compiler/share/classes/com/sun/tools/javac/resources/compiler_de.properties
>> line 409:
>>
>>> 407: compiler.err.unconditional.pattern.and.default=Switch umfasst sowohl
>>> ein nicht bedingtes Muster als auch ein Standardlabel
>
On Mon, 10 Jun 2024 22:48:13 GMT, Damon Nguyen wrote:
>> src/jdk.jdi/share/classes/com/sun/tools/example/debug/tty/TTYResources_de.java
>> line 325:
>>
>>> 323: {"Unexpected event type", "Unerwarteter Ereignistyp: {0}"},
>>> 324: {"unknown", "unbekannt"},
>>> 325: {"Unmo
On Tue, 11 Jun 2024 05:43:12 GMT, Christian Stein wrote:
>> src/java.desktop/share/classes/sun/print/resources/serviceui_de.properties
>> line 66:
>>
>>> 64: label.size=G&röße:
>>> 65: label.source=&Quelle:
>>> 66: label.outputbins=A&usgabefächer:
>>
>> How confident are we in this ?
>> google
On Mon, 10 Jun 2024 20:19:51 GMT, Phil Race wrote:
>> Damon Nguyen has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Remove double quotes
>
> src/java.desktop/share/classes/sun/print/resources/serviceui_de.properties
> line 66:
>
>> 64:
> After PR https://github.com/openjdk/jdk/pull/16245, C2 optimizes stores into
> primitive arrays by combining values into larger stores.
>
> This PR rewrites the code of appendNull and append(boolean) methods so that
> these two methods can be optimized by C2.
Shaojin Wen has updated the pul
> After PR https://github.com/openjdk/jdk/pull/16245, C2 optimizes stores into
> primitive arrays by combining values into larger stores.
>
> This PR rewrites the code of appendNull and append(boolean) methods so that
> these two methods can be optimized by C2.
Shaojin Wen has updated the pul
> This patch changes the algorithm of `Node::dominates` to make the result more
> precise, and allows the iterators of `ConcurrentHashMap` to be scalar
> replaced.
>
> The previous algorithm will return a conservative result when encountering a
> dead control flow, and only try the first two in
On Mon, 10 Jun 2024 13:36:06 GMT, Robert Toyonaga wrote:
> But what about when an int is passed to isspace?
The current casts to int are incorrect as a negative value would be
sign-extended and then fail the range check. I think we have to cast to
unsigned char in all cases in the caller, whic
On Mon, 10 Jun 2024 02:31:00 GMT, David Holmes wrote:
> Sets the version to 24/24-ea and the copyright year to 2025.
>
> Content changes related to the start of release (e.g. for removed options in
> java.1) are handled separately.
>
> This initial generation also picks up the unpublished chan
On Mon, 10 Jun 2024 17:27:18 GMT, Iris Clark wrote:
>> David Holmes has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Regenerated after merge
>
> Marked as reviewed by iris (Reviewer).
Thanks for the review @irisclark .
I've merged in th
> Sets the version to 24/24-ea and the copyright year to 2025.
>
> Content changes related to the start of release (e.g. for removed options in
> java.1) are handled separately.
>
> This initial generation also picks up the unpublished changes from:
>
> - JDK-8330807: Update Manpage for obsole
> Sets the version to 24/24-ea and the copyright year to 2025.
>
> Content changes related to the start of release (e.g. for removed options in
> java.1) are handled separately.
>
> This initial generation also picks up the unpublished changes from:
>
> - JDK-8330807: Update Manpage for obsole
> WIP
>
> This changeset contains hsdis for Windows/gcc Port. It supports both the
> binutils and capstone backends, though the LLVM backend is left out due to
> compatibility issues encountered during the build. Currently, which gcc
> distributions are supported is still to be clarified, as se
On Mon, 10 Jun 2024 12:12:58 GMT, Shaojin Wen wrote:
> After PR https://github.com/openjdk/jdk/pull/16245, C2 optimizes stores into
> primitive arrays by combining values into larger stores.
>
> This PR rewrites the code of appendNull and append(boolean) methods so that
> these two methods c
On Mon, 10 Jun 2024 12:12:58 GMT, Shaojin Wen wrote:
> After PR https://github.com/openjdk/jdk/pull/16245, C2 optimizes stores into
> primitive arrays by combining values into larger stores.
>
> This PR rewrites the code of appendNull and append(boolean) methods so that
> these two methods c
On Mon, 10 Jun 2024 21:38:18 GMT, Damon Nguyen wrote:
>> While we do not translate Java keywords (when used as such, e.g. "`class`
>> not allowed here"), and option names (e.g. `--class-path`) we have typically
>> translated other text that is not mandated by the language or tools, (e.g.
>> `t
On Mon, 10 Jun 2024 14:32:46 GMT, Emanuel Peter wrote:
>> # 1. Compare with unsafe branch
>>
>> 1. current (`5e815`)
>> https://github.com/wenshao/jdk/tree/optim_str_builder_append_202406
>> 2. unsafe (`adc220`)
>> https://github.com/wenshao/jdk/tree/optim_str_builder_append_202406_unsafe
>>
On Mon, 10 Jun 2024 22:19:49 GMT, Chris Plummer wrote:
>> Damon Nguyen has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Remove double quotes
>
> src/jdk.jdi/share/classes/com/sun/tools/example/debug/tty/TTYResources_de.java
> line 325:
>
On Mon, 10 Jun 2024 21:58:29 GMT, Damon Nguyen wrote:
>> This issue is responsible for updating the translations of all the
>> localize(able) resources in the JDK. Primarily, the changes between JDK 22
>> RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated.
>>
>> The tra
> This issue is responsible for updating the translations of all the
> localize(able) resources in the JDK. Primarily, the changes between JDK 22
> RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated.
>
> The translation tool adjusted some definitions, which causes some ch
On Fri, 7 Jun 2024 22:46:44 GMT, Damon Nguyen wrote:
> This issue is responsible for updating the translations of all the
> localize(able) resources in the JDK. Primarily, the changes between JDK 22
> RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated.
>
> The translati
On Mon, 10 Jun 2024 20:12:42 GMT, Jonathan Gibbons wrote:
>> This is what I've determined as well. I planned to leave these untranslated
>> to reduce noise between drops as well.
>
> While we do not translate Java keywords (when used as such, e.g. "`class` not
> allowed here"), and option names
On Mon, 10 Jun 2024 20:46:46 GMT, Naoto Sato wrote:
> Did not look at each for translation accuracy (did some for Japanese), but
> looks good overall. It's great to see diffs in native languages, rather than
> those cryptic `\u` notation!
Justin's tool is responsible for this. Made the pro
On Fri, 7 Jun 2024 22:46:44 GMT, Damon Nguyen wrote:
> This issue is responsible for updating the translations of all the
> localize(able) resources in the JDK. Primarily, the changes between JDK 22
> RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated.
>
> The translati
On Mon, 10 Jun 2024 20:20:52 GMT, Phil Race wrote:
>> This issue is responsible for updating the translations of all the
>> localize(able) resources in the JDK. Primarily, the changes between JDK 22
>> RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated.
>>
>> The transl
On Mon, 10 Jun 2024 20:12:42 GMT, Jonathan Gibbons wrote:
>> This is what I've determined as well. I planned to leave these untranslated
>> to reduce noise between drops as well.
>
> While we do not translate Java keywords (when used as such, e.g. "`class` not
> allowed here"), and option names
On Mon, 10 Jun 2024 20:20:26 GMT, Jonathan Gibbons wrote:
>> This issue is responsible for updating the translations of all the
>> localize(able) resources in the JDK. Primarily, the changes between JDK 22
>> RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated.
>>
>> The
On Fri, 7 Jun 2024 22:46:44 GMT, Damon Nguyen wrote:
> This issue is responsible for updating the translations of all the
> localize(able) resources in the JDK. Primarily, the changes between JDK 22
> RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated.
>
> The translati
On Fri, 7 Jun 2024 22:46:44 GMT, Damon Nguyen wrote:
> This issue is responsible for updating the translations of all the
> localize(able) resources in the JDK. Primarily, the changes between JDK 22
> RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated.
>
> The translati
On Sun, 9 Jun 2024 22:07:44 GMT, Damon Nguyen wrote:
>> src/jdk.jshell/share/classes/jdk/internal/jshell/tool/resources/l10n_de.properties
>> line 183:
>>
>>> 181: jshell.fix.wrong.shortcut =Unerwartetes Zeichen nach
>>> Umschalt+Tab.\nVerwenden Sie "I" für automatischen Import, "V" zur
>>> V
On Sun, 9 Jun 2024 22:49:11 GMT, Justin Lu wrote:
>> This issue is responsible for updating the translations of all the
>> localize(able) resources in the JDK. Primarily, the changes between JDK 22
>> RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated.
>>
>> The transla
On Fri, 7 Jun 2024 22:46:44 GMT, Damon Nguyen wrote:
> This issue is responsible for updating the translations of all the
> localize(able) resources in the JDK. Primarily, the changes between JDK 22
> RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated.
>
> The translati
On Fri, 7 Jun 2024 22:46:44 GMT, Damon Nguyen wrote:
> This issue is responsible for updating the translations of all the
> localize(able) resources in the JDK. Primarily, the changes between JDK 22
> RDP 1 and the integration of the JDK 23 RDP 1 L10n drop will be translated.
>
> The translati
This issue is responsible for updating the translations of all the
localize(able) resources in the JDK. Primarily, the changes between JDK 22 RDP
1 and the integration of the JDK 23 RDP 1 L10n drop will be translated.
The translation tool adjusted some definitions, which causes some changes in
On Mon, 10 Jun 2024 12:29:32 GMT, Pavel Rappo wrote:
> Please review this fix for an intermittent test failure.
>
> On some configurations, the default `expect` timeout of 10 seconds is
> insufficient. It is increased to 20; it's hard to imagine a configuration for
> which that new value would
On Mon, 10 Jun 2024 16:28:39 GMT, Daniel Jeliński wrote:
>> `GetExitCodeProcess` method is not reliable for checking if a process exited
>> already; it returns 259 (STILL_ACTIVE) if the process hasn't exited yet, but
>> the same value is returned when the process exited with code 259. In order
On Mon, 10 Jun 2024 16:24:27 GMT, Daniel Jeliński wrote:
>> Another reason to retain the checking of GetThreadInterruptEvent is to be
>> belt and suspenders against the Java code changing and opening up a
>> potential error. At the moment, the reaper thread is encapsulated and not
>> likely to
> Misc small doc updates and addition of `@Overrides` annotations.
Joe Darcy has updated the pull request with a new target base due to a merge or
a rebase. The incremental webrev excludes the unrelated changes brought in by
the merge/rebase. The pull request contains two additional commits sinc
On Mon, 10 Jun 2024 02:31:00 GMT, David Holmes wrote:
> Sets the version to 24/24-ea and the copyright year to 2025.
>
> Content changes related to the start of release (e.g. for removed options in
> java.1) are handled separately.
>
> This initial generation also picks up the unpublished chan
On Fri, 7 Jun 2024 21:05:33 GMT, Joe Darcy wrote:
> Use the value tag to make the javadoc for various format-related constants
> more informative to readers. Currently the information is available by
> following the "Constant Field Values" link.
>
> I'll reflow the paragraphs before a push.
T
On Mon, 10 Jun 2024 17:09:27 GMT, Jorn Vernee wrote:
> The term 'subset' doesn't feel right to me here, since we're only talking
> about a single memory region (not a set of memory region**s**). I suggest
> 'sub-region' instead.
Actually, nvm, that doesn't work for `reinterpret` since the retu
On Mon, 10 Jun 2024 15:45:07 GMT, Per Minborg wrote:
> This PR proposes to explicitly state that returned segments form the
> `asSlice` and `reinterpret` method share memory regions with `this` memory
> segment.
>
> Note: The term "subset" means a true subset or the same set.
The term 'subse
> Use the value tag to make the javadoc for various format-related constants
> more informative to readers. Currently the information is available by
> following the "Constant Field Values" link.
>
> I'll reflow the paragraphs before a push.
Joe Darcy has updated the pull request incrementally
On Fri, 7 Jun 2024 22:16:36 GMT, Joe Darcy wrote:
>> Use the value tag to make the javadoc for various format-related constants
>> more informative to readers. Currently the information is available by
>> following the "Constant Field Values" link.
>>
>> I'll reflow the paragraphs before a pus
On Mon, 10 Jun 2024 15:30:39 GMT, Per Minborg wrote:
>> This PR proposes to retain the read-only state when any of the
>> `MemorySegment::reinterpret` methods is called.
>>
>> Previously, the read-only state was lost and the returned `MemorySegment`
>> was always writable regardless of the ori
> JMX uses APIs related to the Security Mananger which are deprecated. Use of
> AccessControlContext will be removed when Security Manager is removed.
>
> Until then, updates are needed to not require setting
> -Djava.security.manager=allow to use JMX authentication.
Kevin Walls has updated t
On Fri, 10 Nov 2023 08:17:22 GMT, Per Minborg wrote:
>> This PR outlines a solution for making immutable maps `@ValueBased` by
>> removing cacheing of certain values in `AbstractMap`.
>>
>> By removing these caching fields in `AbstractMap`, we can make the immutable
>> maps `@ValueBased` and a
On Mon, 10 Jun 2024 13:48:08 GMT, Roger Riggs wrote:
>> I don't have a test or counter example and don't have a detailed
>> understanding of the blocking needs of Hotspot.
>
> Another reason to retain the checking of GetThreadInterruptEvent is to be
> belt and suspenders against the Java code c
> `GetExitCodeProcess` method is not reliable for checking if a process exited
> already; it returns 259 (STILL_ACTIVE) if the process hasn't exited yet, but
> the same value is returned when the process exited with code 259. In order to
> check if the process exited, we need to check if its han
On Mon, 10 Jun 2024 12:12:58 GMT, Shaojin Wen wrote:
> After PR https://github.com/openjdk/jdk/pull/16245, C2 optimizes stores into
> primitive arrays by combining values into larger stores.
>
> This PR rewrites the code of appendNull and append(boolean) methods so that
> these two methods c
This PR proposes to explicitly state that returned segments form the `asSlice`
and `reinterpret` method share memory regions with `this` memory segment.
Note: The term "subset" means a true subset or the same set.
-
Commit messages:
- Add segments share the same backing store in d
On Mon, 10 Jun 2024 14:28:25 GMT, Alan Bateman wrote:
>> JMX uses APIs related to the Security Mananger which are deprecated. Use of
>> AccessControlContext will be removed when Security Manager is removed.
>>
>> Until then, updates are needed to not require setting
>> -Djava.security.manage
On Fri, 7 Jun 2024 00:23:10 GMT, Stuart Marks wrote:
>> A documentation-only change to match the original intent and the implemented
>> behavior.
>
> Oh, this needs a CSR too, since it's a change to a normative assertion.
> Should be pretty simple though.
@stuart-marks A [CSR](https://bugs.ope
> This PR proposes to retain the read-only state when any of the
> `MemorySegment::reinterpret` methods is called.
>
> Previously, the read-only state was lost and the returned `MemorySegment` was
> always writable regardless of the original segment's read-only state.
Per Minborg has updated th
On Mon, 10 Jun 2024 15:12:59 GMT, Per Minborg wrote:
> > Note that `asSlice` methods also lacks a similar guarantee on read-only
> > (but the impl works fine there)
>
> Even though not mentioned in the original issue, we could add documentation
> for this here as well. Or maybe via
> https://
On Mon, 10 Jun 2024 13:45:16 GMT, Maurizio Cimadamore
wrote:
> Note that `asSlice` methods also lacks a similar guarantee on read-only (but
> the impl works fine there)
Even though not mentioned in the original issue, we could add documentation for
this here as well. Or maybe via https://bugs
On Mon, 10 Jun 2024 14:59:42 GMT, Per Minborg wrote:
>> This PR proposes to retain the read-only state when any of the
>> `MemorySegment::reinterpret` methods is called.
>>
>> Previously, the read-only state was lost and the returned `MemorySegment`
>> was always writable regardless of the ori
> This PR proposes to retain the read-only state when any of the
> `MemorySegment::reinterpret` methods is called.
>
> Previously, the read-only state was lost and the returned `MemorySegment` was
> always writable regardless of the original segment's read-only state.
Per Minborg has updated th
On Mon, 10 Jun 2024 13:04:08 GMT, Shaojin Wen wrote:
>> After PR https://github.com/openjdk/jdk/pull/16245, C2 optimizes stores into
>> primitive arrays by combining values into larger stores.
>>
>> This PR rewrites the code of appendNull and append(boolean) methods so that
>> these two meth
On Mon, 10 Jun 2024 11:28:28 GMT, Kevin Walls wrote:
> JMX uses APIs related to the Security Mananger which are deprecated. Use of
> AccessControlContext will be removed when Security Manager is removed.
>
> Until then, updates are needed to not require setting
> -Djava.security.manager=allo
On Mon, 10 Jun 2024 11:28:28 GMT, Kevin Walls wrote:
> JMX uses APIs related to the Security Mananger which are deprecated. Use of
> AccessControlContext will be removed when Security Manager is removed.
>
> Until then, updates are needed to not require setting
> -Djava.security.manager=allo
JMX uses APIs related to the Security Mananger which are deprecated. Use of
AccessControlContext will be removed when Security Manager is removed.
Until then, updates are needed to not require setting
-Djava.security.manager=allow to use JMX authentication.
-
Commit messages:
-
On Mon, 10 Jun 2024 12:12:58 GMT, Shaojin Wen wrote:
> After PR https://github.com/openjdk/jdk/pull/16245, C2 optimizes stores into
> primitive arrays by combining values into larger stores.
>
> This PR rewrites the code of appendNull and append(boolean) methods so that
> these two methods c
On Mon, 10 Jun 2024 03:51:40 GMT, Glavo wrote:
>> Things have changed since https://github.com/openjdk/jdk/pull/14636 was
>> closed, so let me reopen it.
>>
>> https://github.com/openjdk/jdk/pull/15386 confirmed that `VarHandle` in BALE
>> caused a startup regression. In order to not have any
On Fri, 7 Jun 2024 21:13:56 GMT, Roger Riggs wrote:
>> Interesting. Could you point me to a related bug / documentation / evidence?
>> I tested the WaitForSingleObject version (commit
>> 6524747a5ebc51c760b14e90309c5f18b58b20e2) on Windows 11, and I didn't
>> observe any problems with shutting
On Mon, 10 Jun 2024 13:18:43 GMT, Per Minborg wrote:
> This PR proposes to retain the read-only state when any of the
> `MemorySegment::reinterpret` methods is called.
>
> Previously, the read-only state was lost and the returned `MemorySegment` was
> always writable regardless of the original
On Mon, 10 Jun 2024 12:12:58 GMT, Shaojin Wen wrote:
> After PR https://github.com/openjdk/jdk/pull/16245, C2 optimizes stores into
> primitive arrays by combining values into larger stores.
>
> This PR rewrites the code of appendNull and append(boolean) methods so that
> these two methods c
After PR https://github.com/openjdk/jdk/pull/16245, C2 optimizes stores into
primitive arrays by combining values into larger stores.
This PR rewrites the code of appendNull and append(boolean) methods so that
these two methods can be optimized by C2.
-
Commit messages:
- Optimi
On Fri, Jun 7, 2024 at 1:19 PM Ron Pressler wrote:
>
> > On 6 Jun 2024, at 18:37, David Lloyd wrote:
> >
> > Just bumping this one more time. I intend to start by opening a JIRA to
> add the two proposed methods to `ReflectionFactory`, and go from there. I
> guess that we might need a JEP for th
On Wed, 5 Jun 2024 20:08:10 GMT, Robert Toyonaga wrote:
> ### Summary
> This change ensures we don't get undefined behavior when
> calling[`isspace`](https://pubs.opengroup.org/onlinepubs/007904975/functions/isspace.html).
> `isspace` accepts an `int` argument that "the application shall ensur
This PR proposes to retain the read-only state when any of the
`MemorySegment::reinterpret` methods is called.
Previously, the read-only state was lost and the returned `MemorySegment` was
always writable regardless of the original segment's read-only state.
-
Commit messages:
- M
> WIP
>
> This changeset contains hsdis for Windows/gcc Port. It supports both the
> binutils and capstone backends, though the LLVM backend is left out due to
> compatibility issues encountered during the build. Currently, which gcc
> distributions are supported is still to be clarified, as se
On Mon, 10 Jun 2024 04:08:41 GMT, Chen Liang wrote:
>> Please review this patch that fixes a critical issue that breaks some Proxy
>> usages.
>>
>> CONSTANT_Class and CONSTANT_MethodType must fail resolution for inaccessible
>> package-private types per JVMS 5.4.3.1 and 5.4.3.5, yet such types
Please review this fix for an intermittent test failure.
On some configurations, the default `expect` timeout of 10 seconds is
insufficient. It is increased to 20; it's hard to imagine a configuration for
which that new value would still be insufficient, but we'll see.
Aside from that, test-gen
On Mon, 10 Jun 2024 04:08:41 GMT, Chen Liang wrote:
>> Please review this patch that fixes a critical issue that breaks some Proxy
>> usages.
>>
>> CONSTANT_Class and CONSTANT_MethodType must fail resolution for inaccessible
>> package-private types per JVMS 5.4.3.1 and 5.4.3.5, yet such types
On Mon, 10 Jun 2024 04:08:41 GMT, Chen Liang wrote:
>> Please review this patch that fixes a critical issue that breaks some Proxy
>> usages.
>>
>> CONSTANT_Class and CONSTANT_MethodType must fail resolution for inaccessible
>> package-private types per JVMS 5.4.3.1 and 5.4.3.5, yet such types
On Thu, 6 Jun 2024 12:46:36 GMT, jengebr wrote:
>> Improve `java/util/concurrent/CopyOnWriteArrayList` by eliminating needless
>> cloning of Object[0] instances. This cloning is intended to prevent callers
>> from changing array contents, but many `CopyOnWriteArrayList`s are allocated
>> to si
On Thu, 6 Jun 2024 14:29:46 GMT, Aleksey Shipilev wrote:
>> `clone()` performs a shallow copy, so there is currently no `Object[]`
>> allocated and therefore nothing to optimize. I don't think there's any work
>> to do here.
>>
>> @DougLea @viktorklang-ora can you confirm?
>
>> clone() perfor
On Tue, 16 Apr 2024 11:47:23 GMT, Per Minborg wrote:
> # Stable Values & Collections (Internal)
>
> ## Summary
> This PR proposes to introduce an internal _Stable Values & Collections_ API,
> which provides immutable value holders where elements are initialized _at
> most once_. Stable Values
On Fri, 17 May 2024 09:31:33 GMT, Per Minborg wrote:
>> # Stable Values & Collections (Internal)
>>
>> ## Summary
>> This PR proposes to introduce an internal _Stable Values & Collections_ API,
>> which provides immutable value holders where elements are initialized _at
>> most once_. Stable V
> Can I please get a review for this patch which proposes to implement the
> enhancement request noted in https://bugs.openjdk.java.net/browse/JDK-8173970?
>
> The commit in this PR introduces the `-o` and `--output-dir` option to the
> `jar` command. The option takes a path to a destination dir
On Fri, 7 Jun 2024 18:58:36 GMT, Claes Redestad wrote:
>> This PR refactors type matching in BootstrapMethodInvoker and adds a few
>> types, seeking to improve bootstrap overheads of some ConstantBootstraps and
>> in particular the ProxyGenerator condys generated for e.g. annotation
>> proxies
> This PR refactors type matching in BootstrapMethodInvoker and adds a few
> types, seeking to improve bootstrap overheads of some ConstantBootstraps and
> in particular the ProxyGenerator condys generated for e.g. annotation proxies
> since [JDK-8332457](https://bugs.openjdk.org/browse/JDK-8332
On Mon, 10 Jun 2024 08:30:59 GMT, Claes Redestad wrote:
> Trivially removing the a leftover, unused `ClassValue` from
> `java.lang.invoke.VarHandle`
This pull request has now been integrated.
Changeset: 7b43a8cd
Author:Claes Redestad
URL:
https://git.openjdk.org/jdk/commit/7b43a8cd
On Mon, 10 Jun 2024 08:57:23 GMT, Maurizio Cimadamore
wrote:
> Good catch!
Thanks!
-
PR Comment: https://git.openjdk.org/jdk/pull/19620#issuecomment-2157910277
On Fri, 7 Jun 2024 18:58:36 GMT, Claes Redestad wrote:
>> This PR refactors type matching in BootstrapMethodInvoker and adds a few
>> types, seeking to improve bootstrap overheads of some ConstantBootstraps and
>> in particular the ProxyGenerator condys generated for e.g. annotation
>> proxies
On Fri, 7 Jun 2024 18:58:36 GMT, Claes Redestad wrote:
>> This PR refactors type matching in BootstrapMethodInvoker and adds a few
>> types, seeking to improve bootstrap overheads of some ConstantBootstraps and
>> in particular the ProxyGenerator condys generated for e.g. annotation
>> proxies
On Mon, 10 Jun 2024 07:56:03 GMT, Claes Redestad wrote:
>> Claes Redestad has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Adress review comments, add ConstantBootstraps#invoke to list of
>> recognized type signatures
>
> Sure. If you ca
On Wed, 5 Jun 2024 20:08:10 GMT, Robert Toyonaga wrote:
> ### Summary
> This change ensures we don't get undefined behavior when
> calling[`isspace`](https://pubs.opengroup.org/onlinepubs/007904975/functions/isspace.html).
> `isspace` accepts an `int` argument that "the application shall ensur
On Mon, 10 Jun 2024 08:30:59 GMT, Claes Redestad wrote:
> Trivially removing the a leftover, unused `ClassValue` from
> `java.lang.invoke.VarHandle`
Good catch!
-
Marked as reviewed by mcimadamore (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/19620#pullrequestreview-210
On Mon, 10 Jun 2024 08:20:38 GMT, David Holmes wrote:
> > "To use these functions safely with plain chars (or signed chars), the
> > argument should first be converted to unsigned char"
>
> @tstuefe wow! Okay. That is a surprise to me. A cast to unsigned char doesn't
> actually do anything. Ev
Trivially removing the a leftover, unused `ClassValue` from
`java.lang.invoke.VarHandle`
-
Commit messages:
- 8333824: Unused ClassValue in VarHandles
Changes: https://git.openjdk.org/jdk/pull/19620/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=19620&range=00
Issue: ht
On Fri, 7 Jun 2024 06:07:17 GMT, Thomas Stuefe wrote:
> "To use these functions safely with plain chars (or signed chars), the
> argument should first be converted to unsigned char"
@tstuefe wow! Okay. That is a surprise to me. A cast to unsigned char doesn't
actually do anything. Every char i
On Sat, 8 Jun 2024 00:19:55 GMT, Shaojin Wen wrote:
> After PR #16245, C2 optimizes stores into primitive arrays by combining
> values into larger stores. In the UUID.toString method,
> ByteArrayLittleEndian can be removed, making the code more elegant and faster.
This pull request has now b
On Mon, 10 Jun 2024 07:32:26 GMT, Shaojin Wen wrote:
>> After PR #16245, C2 optimizes stores into primitive arrays by combining
>> values into larger stores. In the UUID.toString method,
>> ByteArrayLittleEndian can be removed, making the code more elegant and
>> faster.
>
> Shaojin Wen has
On Mon, 10 Jun 2024 07:15:51 GMT, Alan Bateman wrote:
>> Sets the version to 24/24-ea and the copyright year to 2025.
>>
>> Content changes related to the start of release (e.g. for removed options in
>> java.1) are handled separately.
>>
>> This initial generation also picks up the unpublishe
On Fri, 7 Jun 2024 18:58:36 GMT, Claes Redestad wrote:
>> This PR refactors type matching in BootstrapMethodInvoker and adds a few
>> types, seeking to improve bootstrap overheads of some ConstantBootstraps and
>> in particular the ProxyGenerator condys generated for e.g. annotation
>> proxies
On Mon, 10 Jun 2024 07:32:26 GMT, Shaojin Wen wrote:
>> After PR #16245, C2 optimizes stores into primitive arrays by combining
>> values into larger stores. In the UUID.toString method,
>> ByteArrayLittleEndian can be removed, making the code more elegant and
>> faster.
>
> Shaojin Wen has
1 - 100 of 104 matches
Mail list logo