Re: RFR: JDK-8298445: Add LeakSanitizer support in HotSpot [v6]

2023-02-08 Thread David Holmes
On Wed, 8 Feb 2023 13:37:04 GMT, Magnus Ihse Bursie wrote: >> Justin King has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Revert changes to JDK >> >> Signed-off-by: Justin King > > I apologize, the fault lies entirely with me. Just

Withdrawn: 8301854: C4819 warnings were reported in libfreetype on Windows

2023-02-08 Thread Yasumasa Suenaga
On Mon, 6 Feb 2023 12:34:36 GMT, Yasumasa Suenaga wrote: > This is subtask of https://github.com/openjdk/jdk/pull/12427 . > > I have seen C4819 warning in libfreetype files on Windows (CP932: Japanese > locale) > > > d:\github-forked\jdk\src\java.desktop\share\native\libfreetype\src\autofit\a

Re: RFR: 8301854: C4819 warnings were reported in libfreetype on Windows

2023-02-08 Thread Yasumasa Suenaga
On Mon, 6 Feb 2023 12:34:36 GMT, Yasumasa Suenaga wrote: > This is subtask of https://github.com/openjdk/jdk/pull/12427 . > > I have seen C4819 warning in libfreetype files on Windows (CP932: Japanese > locale) > > > d:\github-forked\jdk\src\java.desktop\share\native\libfreetype\src\autofit\a

Re: RFR: 8301855: C4819 warnings were reported in harfbuzz on Windows

2023-02-08 Thread Yasumasa Suenaga
On Mon, 6 Feb 2023 12:35:16 GMT, Yasumasa Suenaga wrote: > This is subtask of https://github.com/openjdk/jdk/pull/12427 . > > I have seen C4819 warning in harfbuzz files on Windows (CP932: Japanese > locale) > > > d:\github-forked\jdk\src\java.desktop\share\native\libharfbuzz\hb.hh(1): > err

Withdrawn: 8301855: C4819 warnings were reported in harfbuzz on Windows

2023-02-08 Thread Yasumasa Suenaga
On Mon, 6 Feb 2023 12:35:16 GMT, Yasumasa Suenaga wrote: > This is subtask of https://github.com/openjdk/jdk/pull/12427 . > > I have seen C4819 warning in harfbuzz files on Windows (CP932: Japanese > locale) > > > d:\github-forked\jdk\src\java.desktop\share\native\libharfbuzz\hb.hh(1): > err

Re: RFR: JDK-8298445: Add LeakSanitizer support in HotSpot [v6]

2023-02-08 Thread Thomas Stuefe
On Wed, 8 Feb 2023 15:39:17 GMT, Justin King wrote: > > I read through your explanation, and through the [design > > docs](https://github.com/google/sanitizers/wiki/AddressSanitizerLeakSanitizerDesignDocument), > > but my question remains unanswered. See below. > > > > Metaspace objects hold po

RE: JDK-8302000: jdk11u build has a very subtle race condition causing builds to break.

2023-02-08 Thread Langer, Christoph
Hi Kumar, yes, I would agree. Option [3] is my preferred. It brings the code closer to upstream. Thanks for tackling this. Cheers Christoph From: build-dev On Behalf Of Kumar Srinivasan Sent: Dienstag, 7. Februar 2023 20:20 To: build-...@openjdk.java.net; jdk-updates-...@openjdk.java.net Subj

Integrated: 8301717: Remove obsolete jib profiles

2023-02-08 Thread Erik Joelsson
On Thu, 2 Feb 2023 22:44:40 GMT, Erik Joelsson wrote: > This patch removes some now obsolete jib profiles configurations. This pull request has now been integrated. Changeset: 8a9e383d Author:Erik Joelsson URL: https://git.openjdk.org/jdk/commit/8a9e383dba58ed047ca46007b5af186aade05

Re: RFR: JDK-8298445: Add LeakSanitizer support in HotSpot [v6]

2023-02-08 Thread Justin King
On Wed, 8 Feb 2023 06:58:52 GMT, Thomas Stuefe wrote: > I read through your explanation, and through the [design > docs](https://github.com/google/sanitizers/wiki/AddressSanitizerLeakSanitizerDesignDocument), > but my question remains unanswered. See below. > > > > Metaspace objects hold point

Re: RFR: JDK-8298445: Add LeakSanitizer support in HotSpot [v6]

2023-02-08 Thread Justin King
On Wed, 8 Feb 2023 06:45:28 GMT, Thomas Stuefe wrote: > The informal rule is that ongoing discussions should be closed and that > nobody strongly objects to a change. Two reviewers are easy to come by. The > point of this rule is that you have a reasonable chance to block changes you > strongl

Re: RFR: 8301753: AppendFile needs to add a new line to be compatible with old make

2023-02-08 Thread Magnus Ihse Bursie
On Tue, 7 Feb 2023 20:56:01 GMT, Leonid Mesnik wrote: > The newline is added to be compatible with old make 3.81 which is still used > on MacOSX. > Fixed actually by dnsimon. @lmesnik If you want, I can take over the issue and solve it correctly. Let me know. - PR: https://git.op

Re: RFR: 8301753: AppendFile needs to add a new line to be compatible with old make

2023-02-08 Thread Erik Joelsson
On Tue, 7 Feb 2023 20:56:01 GMT, Leonid Mesnik wrote: > The newline is added to be compatible with old make 3.81 which is still used > on MacOSX. > Fixed actually by dnsimon. It looks like you are correct Magnus, I didn't think about that. I will see what I can do about solving this properly.

Re: RFR: 8301753: AppendFile needs to add a new line to be compatible with old make

2023-02-08 Thread Magnus Ihse Bursie
On Tue, 7 Feb 2023 20:56:01 GMT, Leonid Mesnik wrote: > The newline is added to be compatible with old make 3.81 which is still used > on MacOSX. > Fixed actually by dnsimon. Possible solutions include: * Stripping any potential newline from the end of the input, and keep unconditionally addin

Re: RFR: 8301753: AppendFile needs to add a new line to be compatible with old make

2023-02-08 Thread Magnus Ihse Bursie
On Tue, 7 Feb 2023 20:56:01 GMT, Leonid Mesnik wrote: > The newline is added to be compatible with old make 3.81 which is still used > on MacOSX. > Fixed actually by dnsimon. I don't think this fix is correct. Look again at the documentation of make 4+ `$(file)` command: "When writing files,

Re: RFR: 8294982: Implementation of Classfile API [v12]

2023-02-08 Thread Adam Sotona
On Wed, 8 Feb 2023 12:19:35 GMT, Adam Sotona wrote: >>> `Signature.ThrowableSig` is a `Signature` and it is a common super of >>> `ClassTypeSig` and `TypeVarSig`. >> >> I really don't follow here. ThrowableSig is a piece of a method signature, >> which starts with "^" and is followed by either

Re: RFR: JDK-8298445: Add LeakSanitizer support in HotSpot [v6]

2023-02-08 Thread Magnus Ihse Bursie
On Tue, 7 Feb 2023 15:40:34 GMT, Justin King wrote: >> Adds initial LSan (LeakSanitizer) support to Hotspot. This setup has been >> used to identify multiple leaks so far. It can run most of the test suite >> except those that also fail with ASan, which is being looked at separately. >> It is

Re: RFR: 8294982: Implementation of Classfile API [v12]

2023-02-08 Thread Adam Sotona
On Wed, 8 Feb 2023 11:07:08 GMT, Maurizio Cimadamore wrote: >> `TypeParam` is not a signature, because it simply is not a signature. >> Per spec: >> >> TypeParameter: >> Identifier ClassBound {InterfaceBound} > >> `Signature.ThrowableSig` is a `Signature` and it is a common super of >> `ClassT

Re: RFR: 8294982: Implementation of Classfile API [v13]

2023-02-08 Thread Maurizio Cimadamore
On Wed, 8 Feb 2023 07:31:15 GMT, Adam Sotona wrote: > Any reference to an official specification of CharacterRangeInfo is > appreciated. Thanks. There's this - which points back to javac code :-) https://github.com/jacoco/jacoco/wiki/CharacterRangeTable - PR: https://git.openjdk.

Re: RFR: 8294982: Implementation of Classfile API [v13]

2023-02-08 Thread Maurizio Cimadamore
On Wed, 8 Feb 2023 11:05:34 GMT, Adam Sotona wrote: > `ofSymbols` is an alternative to `of` when conflicting method parameters. In > such case `of` refers to CP entries and `ofSymbols` refer to independent > symbols describing the objects, like for example `ClassDesc`, > `MethodTypeDesc`, `Pac

Re: RFR: 8294982: Implementation of Classfile API [v12]

2023-02-08 Thread Maurizio Cimadamore
On Wed, 8 Feb 2023 07:24:05 GMT, Adam Sotona wrote: >> **Specification:** >> >> MethodSignature: >> [TypeParameters] ( {JavaTypeSignature} ) Result {ThrowsSignature} >> >> Result: >> JavaTypeSignature >> VoidDescriptor >> >> ThrowsSignature: >> ^ ClassTypeSignature >> ^ TypeVariableSignature

Re: RFR: 8294982: Implementation of Classfile API [v13]

2023-02-08 Thread Adam Sotona
On Tue, 7 Feb 2023 16:07:00 GMT, Maurizio Cimadamore wrote: >> Adam Sotona has updated the pull request incrementally with three additional >> commits since the last revision: >> >> - javadoc fixes >> - obsolete identifiers and unused imports cleanup >> - TypeAnnotation.TypePathComponent cl

Re: RFR: 8294982: Implementation of Classfile API [v13]

2023-02-08 Thread Adam Sotona
On Tue, 7 Feb 2023 16:09:56 GMT, Maurizio Cimadamore wrote: >> Adam Sotona has updated the pull request incrementally with three additional >> commits since the last revision: >> >> - javadoc fixes >> - obsolete identifiers and unused imports cleanup >> - TypeAnnotation.TypePathComponent cl

Re: RFR: 8294982: Implementation of Classfile API [v13]

2023-02-08 Thread Adam Sotona
On Tue, 7 Feb 2023 15:59:04 GMT, Maurizio Cimadamore wrote: >> Adam Sotona has updated the pull request incrementally with three additional >> commits since the last revision: >> >> - javadoc fixes >> - obsolete identifiers and unused imports cleanup >> - TypeAnnotation.TypePathComponent cl

Re: RFR: 8294982: Implementation of Classfile API [v14]

2023-02-08 Thread Adam Sotona
> This is root pull request with Classfile API implementation, tests and > benchmarks initial drop into JDK. > > Following pull requests consolidating JDK class files parsing, generating, > and transforming ([JDK-8294957](https://bugs.openjdk.org/browse/JDK-8294957)) > will chain to this one. >

Re: RFR: 8294982: Implementation of Classfile API [v13]

2023-02-08 Thread Adam Sotona
On Tue, 7 Feb 2023 15:26:12 GMT, Maurizio Cimadamore wrote: >> Adam Sotona has updated the pull request incrementally with three additional >> commits since the last revision: >> >> - javadoc fixes >> - obsolete identifiers and unused imports cleanup >> - TypeAnnotation.TypePathComponent cl

Re: RFR: 8294982: Implementation of Classfile API [v13]

2023-02-08 Thread Adam Sotona
On Tue, 7 Feb 2023 15:18:17 GMT, Maurizio Cimadamore wrote: >> Adam Sotona has updated the pull request incrementally with three additional >> commits since the last revision: >> >> - javadoc fixes >> - obsolete identifiers and unused imports cleanup >> - TypeAnnotation.TypePathComponent cl

Re: RFR: 8294982: Implementation of Classfile API [v12]

2023-02-08 Thread Adam Sotona
On Fri, 3 Feb 2023 18:33:46 GMT, Maurizio Cimadamore wrote: >> Adam Sotona has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Classfile API moved under jdk.internal.classfile package > > src/java.base/share/classes/jdk/internal/classfile/F

Re: RFR: 8294982: Implementation of Classfile API [v12]

2023-02-08 Thread Adam Sotona
On Tue, 7 Feb 2023 11:53:46 GMT, Maurizio Cimadamore wrote: >> `BootstrapMethodEntry` is not a constant pool entry, but >> `BootstrapMethodsAttribute` entry. >> It might be rather moved under `attribute` package and renamed to >> `BootstrapMethodInfo` to follow the same pattern as other attrib