On Fri, 20 Dec 2024 13:17:17 GMT, Coleen Phillimore wrote:
>> Please review this change that makes AccessFlags and modifier_flags u2 types
>> and removes the last remnants of Hotspot adding internal access flags. This
>> change moves AccessFlags and modifier_flags in Klass to alignment gaps
>
On Wed, 25 Dec 2024 02:34:16 GMT, Qizheng Xing wrote:
>> This patch fixes unmatched brackets in some source files.
>
> Qizheng Xing has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Revert fix in the CTW Makefile.
Looks good.
-
On Mon, 23 Dec 2024 19:42:34 GMT, Kim Barrett wrote:
>> I strongly suggest avoiding omnibus changes like this when possible (which it
>> is here). Just because it's all about one "kind" of change doesn't make it a
>> cohesive change. This is touching many distinct areas of the JDK, and several
>>
> This patch fixes unmatched brackets in some source files.
Qizheng Xing has updated the pull request incrementally with one additional
commit since the last revision:
Revert fix in the CTW Makefile.
-
Changes:
- all: https://git.openjdk.org/jdk/pull/22861/files
- new: https:
On Tue, 24 Dec 2024 22:22:28 GMT, Kim Barrett wrote:
>> Only this small piece would be appropriate for backport to 24. I suggest
>> splitting this out into its own bug,
>> fixing it in isolation, and backporting that.
>
> The purpose of this Makefile seems to be to build ctw.jar. So far as I ca
On Tue, 24 Dec 2024 22:55:12 GMT, Phil Race wrote:
> I'd like to problem list two tests which have started failing very frequently
This pull request has now been integrated.
Changeset: 4fc445d1
Author:Phil Race
URL:
https://git.openjdk.org/jdk/commit/4fc445d12b4dabd5ce3a6deb23ca6e4f
On Tue, 24 Dec 2024 22:55:12 GMT, Phil Race wrote:
> I'd like to problem list two tests which have started failing very frequently
Looks good and trivial. Thanks for fixing this!
-
Marked as reviewed by lmesnik (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/22877#pullrequ
On Mon, 23 Dec 2024 18:43:07 GMT, Simon Tooke wrote:
> This test fixes an issue with incomplete Windows threads not yet having a
> stack. A test for a null stack_base is added to guard against the potential
> null dereference. An additional test using ZGC is added to the jtreg
> SystemMapTes
I'd like to problem list two tests which have started failing very frequently
-
Commit messages:
- 8346829
Changes: https://git.openjdk.org/jdk/pull/22877/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=22877&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8346829
S
On Tue, 24 Dec 2024 22:55:12 GMT, Phil Race wrote:
> I'd like to problem list two tests which have started failing very frequently
bit of a guess ...
-
PR Comment: https://git.openjdk.org/jdk/pull/22877#issuecomment-2561490332
On Tue, 24 Dec 2024 03:26:23 GMT, Qizheng Xing wrote:
>> This patch fixes unmatched brackets in some source files.
>
> Qizheng Xing has updated the pull request incrementally with three additional
> commits since the last revision:
>
> - Update `hotspot-unit-tests.md` and HTML (using Pandoc 2.
On Tue, 24 Dec 2024 20:27:37 GMT, Kim Barrett wrote:
>> Hmm, apparently this is not run as part of tier 1-3, which I ran in my
>> patch. I wonder how this test is run. And it means this patch needs to be
>> backported to 24.
>
> Only this small piece would be appropriate for backport to 24.
On Tue, 24 Dec 2024 15:45:24 GMT, Chen Liang wrote:
>> I've tested this by running `make`, and the previous Makefile reports an
>> error like file `test/lib/jtreg/SkippedException.java)` not found. So this
>> fix is necessary.
>>
>> I also checked the git log, this change was introduced in
>>
This test fixes an issue with incomplete Windows threads not yet having a
stack. A test for a null stack_base is added to guard against the potential
null dereference. An additional test using ZGC is added to the jtreg
SystemMapTest.
-
Commit messages:
- Merge branch 'pr_8346717
On Tue, 24 Dec 2024 03:26:23 GMT, Qizheng Xing wrote:
>> This patch fixes unmatched brackets in some source files.
>
> Qizheng Xing has updated the pull request incrementally with three additional
> commits since the last revision:
>
> - Update `hotspot-unit-tests.md` and HTML (using Pandoc 2.
On Tue, 24 Dec 2024 03:28:31 GMT, Qizheng Xing wrote:
>> test/hotspot/jtreg/testlibrary/ctw/Makefile line 50:
>>
>>> 48: $(TESTLIBRARY_DIR)/jdk/test/lib/util \
>>> 49: $(TESTLIBRARY_DIR)/jtreg \
>>> 50: -maxdepth 1 -name '*.java')
>>
>> I wonder why this hasn't caused problems and b
On Tue, 24 Dec 2024 03:26:23 GMT, Qizheng Xing wrote:
>> This patch fixes unmatched brackets in some source files.
>
> Qizheng Xing has updated the pull request incrementally with three additional
> commits since the last revision:
>
> - Update `hotspot-unit-tests.md` and HTML (using Pandoc 2.
17 matches
Mail list logo
