RFR: 8354547: REDO: Force clients to explicitly pass mem_tag value, even if it is mtNone [v12]
Gerard Ziemski
gziemski at openjdk.org
Wed Apr 30 16:12:52 UTC 2025
On Wed, 30 Apr 2025 15:53:13 GMT, Gerard Ziemski <gziemski at openjdk.org> wrote:
>> Redo for a follow-up to https://github.com/openjdk/jdk/pull/21843. Here we are focusing on removing the mem tag paremeter with default value of mtNone, to force everyone to provide mem tag, if known.
>>
>> I tried to fill in tag, when I was pretty certain that I had the right type.
>>
>> At least one more follow-up will be needed after this, to change the remaining mtNone to valid values.
>>
>> Testing: passes tier1 locally on my mac, Mach5 tier1-5 in progress
>
> Gerard Ziemski 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 13 additional commits since the last revision:
>
> - Merge remote-tracking branch 'upstream/master' into JDK-8354547
> - Merge remote-tracking branch 'upstream/master' into JDK-8354547
> - Merge remote-tracking branch 'upstream/master' into JDK-8354547
> - after merge fixes
> - Merge remote-tracking branch 'upstream/master' into JDK-8354547
> - Merge remote-tracking branch 'upstream/master' into JDK-8354547
> - Merge remote-tracking branch 'upstream/master' into JDK-8354547
> - Merge remote-tracking branch 'upstream/master' into JDK-8354547
> - Merge remote-tracking branch 'upstream/master' into JDK-8354547
> - remove last default argument (found by Stefan)
> - ... and 3 more: https://git.openjdk.org/jdk/compare/08f4670b...7ca990ff
Mdash looks good. Just merged and ran tier1 locally on macOS. I ran tier1-3 yesterday (1 unrelated failure), so as soon as the checks here are done, I plan on integrating.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/24634#issuecomment-2842509900
More information about the hotspot-dev
mailing list