On Wed, 5 Feb 2025 22:32:58 GMT, Calvin Cheung wrote:
> This changset refactors CDS class paths and module paths validation code into
> a new class `AOTCodeSource` and related class `AOTCodeSourceConfig`. Code has
> been moved from filemap.[c|h]pp, classLoader.[c|h]pp, and
> classLoaderExt.[c|
On Mon, 4 Nov 2024 03:15:25 GMT, Ioi Lam wrote:
>> This is an implementation of [JEP 483: Ahead-of-Time Class Loading &
>> Linking](https://openjdk.org/jeps/483).
>>
>>
>> Note: this is a combined PR of the following individual PRs
>> - https://github.com/openjdk/jdk/pull/20516
>> - https:
On Fri, 25 Oct 2024 18:08:17 GMT, Ashutosh Mehra wrote:
>> Ioi Lam has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - 8342907: Implement AOT testing mode for jtreg tests (authored by
>> @katyapav)
>
On Thu, 24 Oct 2024 03:01:54 GMT, Ioi Lam wrote:
>> This is an implementation of [JEP 483: Ahead-of-Time Class Loading &
>> Linking](https://openjdk.org/jeps/483).
>>
>>
>> Note: this is a combined PR of the following individual PRs
>> - https://github.com/openjdk/jdk/pull/20516
>> - https
On Thu, 24 Oct 2024 03:01:54 GMT, Ioi Lam wrote:
>> This is an implementation of [JEP 483: Ahead-of-Time Class Loading &
>> Linking](https://openjdk.org/jeps/483).
>>
>>
>> Note: this is a combined PR of the following individual PRs
>> - https://github.com/openjdk/jdk/pull/20516
>> - https
On Thu, 24 Oct 2024 03:01:54 GMT, Ioi Lam wrote:
>> This is an implementation of [JEP 483: Ahead-of-Time Class Loading &
>> Linking](https://openjdk.org/jeps/483).
>>
>>
>> Note: this is a combined PR of the following individual PRs
>> - https://github.com/openjdk/jdk/pull/20516
>> - https
On Thu, 19 Sep 2024 04:19:18 GMT, Ioi Lam wrote:
>> This is the 3rd PR for [JEP 483: Ahead-of-Time Class Loading &
>> Linking](https://bugs.openjdk.org/browse/JDK-8315737).
>>
>> **Overview**
>>
>> - A new `-XX:+AOTClassLinking` flag is added. See [JEP
>> 498](https://bugs.openjdk.org/browse/
On Mon, 16 Sep 2024 21:54:49 GMT, Ioi Lam wrote:
>> This is the 3rd PR for [JEP 483: Ahead-of-Time Class Loading &
>> Linking](https://bugs.openjdk.org/browse/JDK-8315737).
>>
>> **Overview**
>>
>> - A new `-XX:+AOTClassLinking` flag is added. See [JEP
>> 498](https://bugs.openjdk.org/browse/
On Mon, 16 Sep 2024 21:54:49 GMT, Ioi Lam wrote:
>> This is the 3rd PR for [JEP 483: Ahead-of-Time Class Loading &
>> Linking](https://bugs.openjdk.org/browse/JDK-8315737).
>>
>> **Overview**
>>
>> - A new `-XX:+AOTClassLinking` flag is added. See [JEP
>> 498](https://bugs.openjdk.org/browse/
On Sat, 7 Sep 2024 00:30:34 GMT, Ioi Lam wrote:
>> I've taken an initial look through but there is an awful lot to try and
>> digest here. I've flagged numerous typos and minor nits.
>>
>> One general query: does this stuff work if the user defines their own
>> initial application classloader?
On Thu, 12 Sep 2024 21:43:24 GMT, Ioi Lam wrote:
>> This is the 3rd PR for [JEP 483: Ahead-of-Time Class Loading &
>> Linking](https://bugs.openjdk.org/browse/JDK-8315737).
>>
>> **Overview**
>>
>> - A new `-XX:+AOTClassLinking` flag is added. See [JEP
>> 498](https://bugs.openjdk.org/browse/
On Wed, 11 Sep 2024 20:26:51 GMT, Ioi Lam wrote:
>> This is the 3rd PR for [JEP 483: Ahead-of-Time Class Loading &
>> Linking](https://bugs.openjdk.org/browse/JDK-8315737).
>>
>> **Overview**
>>
>> - A new `-XX:+AOTClassLinking` flag is added. See [JEP
>> 498](https://bugs.openjdk.org/browse/
On Tue, 10 Sep 2024 22:17:00 GMT, Ashutosh Mehra wrote:
> I will continue my review tomorrow.
My review is done now.
-
PR Comment: https://git.openjdk.org/jdk/pull/20843#issuecomment-2346553992
On Tue, 10 Sep 2024 00:53:32 GMT, Ioi Lam wrote:
>> This is the 3rd PR for [JEP 483: Ahead-of-Time Class Loading &
>> Linking](https://bugs.openjdk.org/browse/JDK-8315737).
>>
>> **Overview**
>>
>> - A new `-XX:+AOTClassLinking` flag is added. See [JEP
>> 498](https://bugs.openjdk.org/browse/
On Tue, 10 Sep 2024 00:53:32 GMT, Ioi Lam wrote:
>> This is the 3rd PR for [JEP 483: Ahead-of-Time Class Loading &
>> Linking](https://bugs.openjdk.org/browse/JDK-8315737).
>>
>> **Overview**
>>
>> - A new `-XX:+AOTClassLinking` flag is added. See [JEP
>> 498](https://bugs.openjdk.org/browse/
On Wed, 24 Jul 2024 10:45:05 GMT, Thomas Stuefe wrote:
>> Some minor improvements to CompilationMemoryStatistic. More details are in
>> [JDK-8337031](https://bugs.openjdk.org/browse/JDK-8337031)
>>
>> Testing:
>> test/hotspot/jtreg/compiler/print/CompileCommandPrintMemStat.java
>>
>> test/
On Tue, 23 Jul 2024 21:46:50 GMT, Ashutosh Mehra wrote:
> Some minor improvements to CompilationMemoryStatistic. More details are in
> [JDK-8337031](https://bugs.openjdk.org/browse/JDK-8337031)
>
> Testing:
> test/hotspot/jtreg/compiler/print/CompileCommandPrintMemStat.j
On Wed, 24 Jul 2024 10:45:05 GMT, Thomas Stuefe wrote:
>> Some minor improvements to CompilationMemoryStatistic. More details are in
>> [JDK-8337031](https://bugs.openjdk.org/browse/JDK-8337031)
>>
>> Testing:
>> test/hotspot/jtreg/compiler/print/CompileCommandPrintMemStat.java
>>
>> test/
ability/dcmd/compiler/CompilerMemoryStatisticTest.java
Ashutosh Mehra has updated the pull request incrementally with one additional
commit since the last revision:
Rename ArenaTagsCounter to ArenaCountersByTag
Signed-off-by: Ashutosh Mehra
-
Changes:
- all: https://git.openjdk.org/jdk/pul
On Tue, 30 Jul 2024 05:18:17 GMT, Thomas Stuefe wrote:
>> Ashutosh Mehra has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Address review comments by Thomas S.
>>
>> Signed-off-by: Ashutosh M
On Sat, 27 Jul 2024 05:44:14 GMT, Thomas Stuefe wrote:
>> What do you mean by x macro? Do you have an example that shows the use of x
>> macro?
>
> You use them already in your patch.
>
> E.g.
>
>
> #define XX(name, whatever, desc) st->print_cr(" " LEGEND_KEY_FMT ": " #name
> #desc
> DO_AR
On Wed, 24 Jul 2024 10:45:05 GMT, Thomas Stuefe wrote:
>> Some minor improvements to CompilationMemoryStatistic. More details are in
>> [JDK-8337031](https://bugs.openjdk.org/browse/JDK-8337031)
>>
>> Testing:
>> test/hotspot/jtreg/compiler/print/CompileCommandPrintMemStat.java
>>
>> test/
ability/dcmd/compiler/CompilerMemoryStatisticTest.java
Ashutosh Mehra has updated the pull request incrementally with one additional
commit since the last revision:
Address review comments by Thomas S.
Signed-off-by: Ashutosh Mehra
-
Changes:
- all: https://git.openjdk.org/jdk/pull/20304
On Fri, 26 Jul 2024 06:08:03 GMT, Thomas Stuefe wrote:
>> Some minor improvements to CompilationMemoryStatistic. More details are in
>> [JDK-8337031](https://bugs.openjdk.org/browse/JDK-8337031)
>>
>> Testing:
>> test/hotspot/jtreg/compiler/print/CompileCommandPrintMemStat.java
>>
>> test/
On Tue, 23 Jul 2024 21:46:50 GMT, Ashutosh Mehra wrote:
> Some minor improvements to CompilationMemoryStatistic. More details are in
> [JDK-8337031](https://bugs.openjdk.org/browse/JDK-8337031)
>
> Testing:
> test/hotspot/jtreg/compiler/print/CompileCommandPrintMemStat.j
Some minor improvements to CompilationMemoryStatistic. More details are in
[JDK-8337031](https://bugs.openjdk.org/browse/JDK-8337031)
Testing:
test/hotspot/jtreg/compiler/print/CompileCommandPrintMemStat.java
test/hotspot/jtreg/serviceability/dcmd/compiler/CompilerMemoryStatisticTest.java
-
On Thu, 28 Sep 2023 23:40:58 GMT, Chris Plummer wrote:
>> Please review this change to fix the operands of the bytecodes that operate
>> on fields when dumping a class using SA.
>>
>> Testing: hotspot_serviceability
>>
>> I have also verified that
>> `test/hotspot/jtreg/serviceability/sa/Clhs
On Thu, 28 Sep 2023 21:23:25 GMT, Ashutosh Mehra wrote:
> Please review this change to fix the operands of the bytecodes that operate
> on fields when dumping a class using SA.
>
> Testing: hotspot_serviceability
>
> I have also verified that
> `test/hotspot/jt
On Thu, 28 Sep 2023 22:32:29 GMT, Chris Plummer wrote:
>> Please review this change to fix the operands of the bytecodes that operate
>> on fields when dumping a class using SA.
>>
>> Testing: hotspot_serviceability
>>
>> I have also verified that
>> `test/hotspot/jtreg/serviceability/sa/Clhs
Please review this change to fix the operands of the bytecodes that operate on
fields when dumping a class using SA.
Testing: hotspot_serviceability
I have also verified that
`test/hotspot/jtreg/serviceability/sa/ClhsdbDumpclass.javaClhsdbDumpclass.java`,
which is in the problem list because o
On Wed, 27 Sep 2023 20:45:00 GMT, Ashutosh Mehra wrote:
> Please review trivial fix to display class name in the output of "where"
> command of SA.
>
> Testing: hotspot_serviceability
This pull request has now been integrated.
Changeset: 065203d4
Author:Ashutosh Me
On Thu, 28 Sep 2023 01:57:59 GMT, David Holmes wrote:
>> Please review trivial fix to display class name in the output of "where"
>> command of SA.
>>
>> Testing: hotspot_serviceability
>
> Has the change been tested in the HSDB GUI as requested in the JBS issue?
@dholmes-ora thanks for review
On Wed, 27 Sep 2023 20:45:00 GMT, Ashutosh Mehra wrote:
> Please review trivial fix to display class name in the output of "where"
> command of SA.
>
> Testing: hotspot_serviceability
I believe my github user name has been linked to the OpenJDK user name. So I am
goi
On Thu, 28 Sep 2023 01:57:59 GMT, David Holmes wrote:
>> Please review trivial fix to display class name in the output of "where"
>> command of SA.
>>
>> Testing: hotspot_serviceability
>
> Has the change been tested in the HSDB GUI as requested in the JBS issue?
@dholmes-ora thanks for pointi
On Wed, 27 Sep 2023 21:07:18 GMT, Chris Plummer wrote:
>> Please review trivial fix to display class name in the output of "where"
>> command of SA.
>>
>> Testing: hotspot_serviceability
>
> Can you add to the CR a copy of the `where` output after this fix? Just a
> short snippet equivalent to
On Wed, 27 Sep 2023 21:07:18 GMT, Chris Plummer wrote:
>> Please review trivial fix to display class name in the output of "where"
>> command of SA.
>>
>> Testing: hotspot_serviceability
>
> Can you add to the CR a copy of the `where` output after this fix? Just a
> short snippet equivalent to
On Wed, 27 Sep 2023 20:45:00 GMT, Ashutosh Mehra wrote:
> Please review trivial fix to display class name in the output of "where"
> command of SA.
>
> Testing: hotspot_serviceability
Updated CR with the output of `where` command after the fix.
--
Please review trivial fix to display class name in the output of "where"
command of SA.
Testing: hotspot_serviceability
-
Commit messages:
- 8313631: SA: stack trace printed using "where" command does not show class
name
Changes: https://git.openjdk.org/jdk/pull/15952/files
Webr
On Mon, 24 Jul 2023 22:12:28 GMT, Ashutosh Mehra wrote:
> This patch adds NestHost and NestMembers attributes to the class dumped by SA.
>
> Testing: `test/hotspot/jtreg/serviceability/sa` and `test/jdk/sun/tools/jhsdb`
> Manual testing by dumping `j.l.String` and
&
On Mon, 24 Jul 2023 22:12:28 GMT, Ashutosh Mehra wrote:
> This patch adds NestHost and NestMembers attributes to the class dumped by SA.
>
> Testing: `test/hotspot/jtreg/serviceability/sa` and `test/jdk/sun/tools/jhsdb`
> Manual testing by dumping `j.l.String` and
&
On Thu, 3 Aug 2023 15:59:14 GMT, Chris Plummer wrote:
>> @ashu-mehra thanks for doing the additional testing. Pity there is no
>> regression/functional test for this.
>
>> @dholmes-ora @plummercj I have improved [dumpclass
>> tests](https://github.com/openjdk/jdk/commit/97618cc9bbadce4b51fc0fbe
On Thu, 3 Aug 2023 15:59:14 GMT, Chris Plummer wrote:
>> @ashu-mehra thanks for doing the additional testing. Pity there is no
>> regression/functional test for this.
>
>> @dholmes-ora @plummercj I have improved [dumpclass
>> tests](https://github.com/openjdk/jdk/commit/97618cc9bbadce4b51fc0fbe
On Wed, 2 Aug 2023 05:07:31 GMT, David Holmes wrote:
> Pity there is no regression/functional test for this.
@dholmes-ora @plummercj I have improved [dumpclass
tests](https://github.com/openjdk/jdk/commit/97618cc9bbadce4b51fc0fbee93557f4dcc8d26a)
to cover up some cases for this PR and
[JDK-8
On Tue, 1 Aug 2023 08:33:24 GMT, David Holmes wrote:
>> @dholmes-ora I confirmed there is no nest-host or nest-members attributes
>> generated by this patch for a top level class which doesn't have any
>> nest-members. Is that what you wanted to verify?
>
> @ashu-mehra That was one case. I also
On Thu, 27 Jul 2023 22:39:03 GMT, David Holmes wrote:
>> @dholmes-ora sorry for responding late. I got sidetracked by some other work.
>>
>>> We need to be sure this works as expected for top-level classes that have
>>> no nest members, and deeply nested nest members, plus dynamically injected
On Tue, 25 Jul 2023 05:36:15 GMT, David Holmes wrote:
>> This patch adds NestHost and NestMembers attributes to the class dumped by
>> SA.
>>
>> Testing: `test/hotspot/jtreg/serviceability/sa` and
>> `test/jdk/sun/tools/jhsdb`
>> Manual testing by dumping `j.l.String` and
>> `j.l.String$CaseI
This patch adds NestHost and NestMembers attributes to the class dumped by SA.
Testing: `test/hotspot/jtreg/serviceability/sa` and `test/jdk/sun/tools/jhsdb`
Manual testing by dumping `j.l.String` and
`j.l.String$CaseInsensitiveComparator` classes.
`j.l.String` shows one entry in `NestMembers` at
On Wed, 12 Jul 2023 19:48:52 GMT, Ashutosh Mehra wrote:
> Please review this fix to correctly read a long value in the runtime constant
> pool. Details are mentioned in the issue [0].
> As an example, before this fix the long value generated by SA's
On Mon, 17 Jul 2023 22:59:30 GMT, David Holmes wrote:
> Approved, but please file that follow up issue.
Follow up issue for removing the dead code -
https://bugs.openjdk.org/browse/JDK-8312232
-
PR Comment: https://git.openjdk.org/jdk/pull/14855#issuecomment-1640234531
On Mon, 17 Jul 2023 02:37:38 GMT, David Holmes wrote:
> I think we need a follow-up RFE to get rid of buildLongFromIntsPD and any
> other dead code.
okay.
@dholmes-ora can you please approve it if there are no other concerns.
-
PR Comment: https://git.openjdk.org/jdk/pull/14855#i
On Fri, 14 Jul 2023 03:08:50 GMT, David Holmes wrote:
> First, what is it about constructing the long from two ints that is incorrect?
The incorrect part is fetching the ints from index and index+1. This doesn't
work for 64-bit platforms because the single entry in runtime constant pool is
lar
On Fri, 30 Jun 2023 14:32:03 GMT, Ashutosh Mehra wrote:
> Please review this PR that enables ClassWriter to write annotations to the
> class file being dumped.
>
> The fields annotations are stored in `Annotations::_fields_annotations` which
> is of type `Array*>`. There
On Wed, 12 Jul 2023 23:21:12 GMT, Chris Plummer wrote:
>> Ashutosh Mehra has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> More review comments
>>
>> Signed-off-by: Ashutosh Mehra
>
>
On Wed, 12 Jul 2023 19:48:52 GMT, Ashutosh Mehra wrote:
> Please review this fix to correctly read a long value in the runtime constant
> pool. Details are mentioned in the issue [0].
> As an example, before this fix the long value generated by SA's
On Wed, 12 Jul 2023 15:17:10 GMT, Thomas Stuefe wrote:
>>> What would be needed to make the Annotations appear in the "printall"
>>> command? I was somehow expecting to see at least something like
>>> "Annotation@".
>>
>> I am not sure what all details `printall` is expected to emit out. L
Please review this fix to correctly read a long value in the runtime constant
pool. Details are mentioned in the issue [0].
Before this fix the long value generated by SA's dumpclass for
java.lang.String.serialVersionUID was:
private static final long serialVersionUID;
descriptor: J
fl
On Tue, 11 Jul 2023 20:33:59 GMT, Chris Plummer wrote:
>> Ashutosh Mehra has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Some code motion and factoring out common code
>>
>> Signed-off-by: Ashut
On Tue, 11 Jul 2023 20:28:29 GMT, Chris Plummer wrote:
>> I think VMObjectFactory is a better place to implement the caching behavior
>> so that all such patterns can benefit from it. I think it is better
>> addressed in another task.
>
> I think maybe you misunderstood what I meant by "cache".
viceability/sa` and `test/jdk/sun/tools/jhsdb`
> Tested it manually by dumping j.l.String class and comparing the annotations
> with the original class using javap.
> The test case mentioned in
> [JDK-8311101](https://bugs.openjdk.org/browse/JDK-8311101) would provide
> better
On Tue, 11 Jul 2023 06:32:24 GMT, Thomas Stuefe wrote:
>> Ashutosh Mehra has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Some code motion and factoring out common code
>>
>> Signed-off-by: Ashut
viceability/sa` and `test/jdk/sun/tools/jhsdb`
> Tested it manually by dumping j.l.String class and comparing the annotations
> with the original class using javap.
> The test case mentioned in
> [JDK-8311101](https://bugs.openjdk.org/browse/JDK-8311101) would provide
> better
On Tue, 11 Jul 2023 06:39:09 GMT, Thomas Stuefe wrote:
>> src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/oops/ConstMethod.java
>> line 470:
>>
>>> 468: if (hasParameterAnnotations()) {
>>> 469: offset += 1;
>>> 470: }
>>
>> Code here and in other places could be tighte
On Fri, 30 Jun 2023 17:59:15 GMT, Chris Plummer wrote:
>> Ashutosh Mehra has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Review comments
>>
>> Signed-off-by: Ashutosh Mehra
>
> src/ho
viceability/sa` and `test/jdk/sun/tools/jhsdb`
> Tested it manually by dumping j.l.String class and comparing the annotations
> with the original class using javap.
> The test case mentioned in
> [JDK-8311101](https://bugs.openjdk.org/browse/JDK-8311101) would provide
> better
On Tue, 11 Jul 2023 07:41:01 GMT, Thomas Stuefe wrote:
> What would be needed to make the Annotations appear in the "printall"
> command? I was somehow expecting to see at least something like
> "Annotation@".
I am not sure what all details `printall` is expected to emit out. Looking at
t
On Mon, 10 Jul 2023 13:53:36 GMT, Thomas Stuefe wrote:
>> This is a continuation of https://github.com/openjdk/jdk/pull/10085. I
>> closed https://github.com/openjdk/jdk/pull/10085 because it had accumulated
>> too much comment history and got confusing. For a history of this issue, see
>> pre
On Fri, 30 Jun 2023 14:32:03 GMT, Ashutosh Mehra wrote:
> Please review this PR that enables ClassWriter to write annotations to the
> class file being dumped.
>
> The fields annotations are stored in `Annotations::_fields_annotations` which
> is of type `Array*>`. There
On Fri, 7 Jul 2023 13:38:34 GMT, Thomas Stuefe wrote:
> (just noticed the patch adds +666 lines, bad sign, I should add another line
> somewhere).
It also deletes 2 lines so that makes it 664 😉
-
PR Comment: https://git.openjdk.org/jdk/pull/14781#issuecomment-1625515410
Please review this PR that enables ClassWriter to write annotations to the
class file being dumped.
The fields annotations are stored in `Annotations::_fields_annotations` which
is of type `Array*>`. There is no class in SA that can represent it.
I have added ArrayOfU1Array to correspond to the
On Thu, 15 Jun 2023 15:06:54 GMT, Ashutosh Mehra wrote:
> Please review this PR that extends SA to write BootstrapMethods attribute
> when dumping the class files.
>
> Tested it by dumping the class file for java/lang/String and comparing the
> BootstrapMethods attribute sho
On Thu, 29 Jun 2023 14:16:16 GMT, Kevin Walls wrote:
>> Ashutosh Mehra has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Add comment about the layout of operands array in constant pool
>>
>> Sign
On Fri, 16 Jun 2023 18:10:58 GMT, Chris Plummer wrote:
>> Ashutosh Mehra has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Address review comments by plummercj
>>
>> Signed-off-by: Ashutosh M
On Fri, 16 Jun 2023 18:10:58 GMT, Chris Plummer wrote:
>> Ashutosh Mehra has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Address review comments by plummercj
>>
>> Signed-off-by: Ashutosh M
On Thu, 22 Jun 2023 16:10:13 GMT, Ashutosh Mehra wrote:
> I am thinking of a comprehensive test that creates a classfile with specific
> attribute, load it in the VM, dump that class file using SA, then disassemble
> the generated class file to check for the presence of the attribute.
On Thu, 22 Jun 2023 03:21:33 GMT, Serguei Spitsyn wrote:
> Do we have any automatic test coverage for this?
Nope, I don't think there is any test for BootstrapMethods. As mentioned
[here](https://github.com/openjdk/jdk/pull/14556#issuecomment-1601946451) the
existing tests - ClhsdbDumpclass.ja
> Please review this PR that extends SA to write BootstrapMethods attribute
> when dumping the class files.
>
> Tested it by dumping the class file for java/lang/String and comparing the
> BootstrapMethods attribute shown by javap for the original and the dumped
> class.
On Wed, 21 Jun 2023 12:47:10 GMT, Kevin Walls wrote:
>> Ashutosh Mehra has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Address review comments by plummercj
>>
>> Signed-off-by: Ashutosh Me
On Tue, 20 Jun 2023 11:31:40 GMT, Daohan Qu wrote:
> This patch adds `StackMapTable` for the class files generated by `clhsdb`'s
> `buildreplayjars` command. This bug manifests itself during my diagnosing
> [JDK-8309751](https://bugs.openjdk.org/browse/JDK-8309751) and needs to be
> fixed firs
On Fri, 16 Jun 2023 18:10:58 GMT, Chris Plummer wrote:
>> Ashutosh Mehra has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Address review comments by plummercj
>>
>> Signed-off-by: Ashutosh M
On Thu, 15 Jun 2023 20:24:39 GMT, Ashutosh Mehra wrote:
>> Please review this PR that extends SA to write BootstrapMethods attribute
>> when dumping the class files.
>>
>> Tested it by dumping the class file for java/lang/String and comparing the
>> BootstrapMe
On Thu, 15 Jun 2023 23:20:56 GMT, Chris Plummer wrote:
>> Ashutosh Mehra has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Address review comments by plummercj
>>
>> Signed-off-by: Ashutosh Mehra
On Thu, 15 Jun 2023 20:55:12 GMT, Chris Plummer wrote:
> Can you tell me what testing you've done? Would be best to call that out in
> the PR description.
Edited the description to add a comment about the testing. Hope this helps.
-
PR Comment: https://git.openjdk.org/jdk/pull/144
On Thu, 15 Jun 2023 18:15:41 GMT, Chris Plummer wrote:
>> Ashutosh Mehra has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Address review comments by plummercj
>>
>> Signed-off-by: Ashutosh Me
> Please review this PR that extends SA to write BootstrapMethods attribute
> when dumping the class files.
Ashutosh Mehra has updated the pull request incrementally with one additional
commit since the last revision:
Address review comments by plummercj
Signed-off-by: Ashutosh
Please review this PR that extends SA to write BootstrapMethods attribute when
dumping the class files.
-
Commit messages:
- 8309979: BootstrapMethods attribute is missing in class files recreated by SA
Changes: https://git.openjdk.org/jdk/pull/14495/files
Webrev: https://webrevs.
On Mon, 3 Apr 2023 03:32:27 GMT, Ioi Lam wrote:
> This PR combines the "open" and "closed" regions of the CDS archive heap into
> a single region. This significantly simplifies the implementation, making it
> more compatible with non-G1 collectors. There's a net removal of ~1000 lines
> in src
86 matches
Mail list logo
