From jwaters at openjdk.org Tue Nov 1 00:10:27 2022 From: jwaters at openjdk.org (Julian Waters) Date: Tue, 1 Nov 2022 00:10:27 GMT Subject: RFR: 8295884: Implement IDE support for Eclipse [v12] In-Reply-To: References: Message-ID: > Eclipse is a popular and very well-known IDE in the world of Java development, utilized widely in many contexts, by beginners and experienced teams alike. Although a relatively lightweight IDE, it features surprisingly powerful indexing and code analysis capabilities, as well as useful tools, among which are make and Ant integration. While the tools it provides are not always as sophisticated as other IDEs (IntelliJ IDEA will likely come to mind as one such competitor), the simplicity of using it, as well as the reliability of this rugged IDE makes up greatly for the slightly less advanced tooling. Eclipse requires very little starting infrastructure in the workspace for all these features and indexing support as well, which makes it a good candidate for developing on the JDK. > > This enhancement adds 4 extra targets to the make system for generating a basic Eclipse Workspace that provides almost full indexing support for the JDK, with varying levels as desired, from a minimalistic option only including the JVM's sources, to generating a workspace with both Java and C/C++ natures included, which allows for using Eclipse's ability to quickly swap between Java and C/C++ mode to work on both native and Java sources at the same time. Cross Compiling support is available, and in its entirety the change touches very little of the existing make system, barring its own Makefile within the ide subdirectory. > > Indexing capabilities utilizing the enhancement: > java > escape Julian Waters 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 16 additional commits since the last revision: - Merge remote-tracking branch 'upstream/master' into eclipse - Merge remote-tracking branch 'upstream/master' into eclipse - Include extra directories required by HotSpot in HotSpot only workspaces - SINGLE_DOLLAR is not required - Nuke accidental comment not meant to be added - Initial support for Language Settings Providers - Prevent Eclipse from automatically attaching binary parsers to workspaces including native code - Merge - Squash issue with commas - File operations should use MakeIO.gmk - ... and 6 more: https://git.openjdk.org/jdk/compare/807a1ba8...db0e248e ------------- Changes: - all: https://git.openjdk.org/jdk/pull/10853/files - new: https://git.openjdk.org/jdk/pull/10853/files/90a7d061..db0e248e Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=10853&range=11 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10853&range=10-11 Stats: 2403 lines in 87 files changed: 1666 ins; 300 del; 437 mod Patch: https://git.openjdk.org/jdk/pull/10853.diff Fetch: git fetch https://git.openjdk.org/jdk pull/10853/head:pull/10853 PR: https://git.openjdk.org/jdk/pull/10853 From jwaters at openjdk.org Tue Nov 1 00:12:00 2022 From: jwaters at openjdk.org (Julian Waters) Date: Tue, 1 Nov 2022 00:12:00 GMT Subject: RFR: 8296115: Allow for compiling the JDK with strict standards conformance [v2] In-Reply-To: References: Message-ID: > [JDK-8241499](https://bugs.openjdk.org/browse/JDK-8241499) proposes to set the -permissive- flag for the Microsoft Visual C++ compiler, to enforce strict standards conformance during compilation, making native code behave more strictly. While adding it to default builds is likely not practical given how much testing is required, as an option it can prove helpful in finding areas of native code that are not conformant to the standard. Instead of applying this to just one compiler, we can also include this for every compiler that has support for such a strict mode, which this change does. Julian Waters 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 four additional commits since the last revision: - Merge remote-tracking branch 'upstream/master' into permissive - Format - Keep docs up to date - New --enable-conforming-compilation option ------------- Changes: - all: https://git.openjdk.org/jdk/pull/10912/files - new: https://git.openjdk.org/jdk/pull/10912/files/08aae161..7a527bc8 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=10912&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10912&range=00-01 Stats: 2196 lines in 78 files changed: 1497 ins; 280 del; 419 mod Patch: https://git.openjdk.org/jdk/pull/10912.diff Fetch: git fetch https://git.openjdk.org/jdk pull/10912/head:pull/10912 PR: https://git.openjdk.org/jdk/pull/10912 From jwaters at openjdk.org Tue Nov 1 00:12:06 2022 From: jwaters at openjdk.org (Julian Waters) Date: Tue, 1 Nov 2022 00:12:06 GMT Subject: RFR: 8295990: Improve make handling of strip flags [v6] In-Reply-To: References: Message-ID: > Strip currently has its flags set by guessing based on the OS, it would be more appropriate to instead set them based on properly checking the strip binary instead of speculating what the correct flags are based on the compiler and OS. Julian Waters 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 17 additional commits since the last revision: - Merge branch 'openjdk:master' into patch-1 - Merge branch 'openjdk:master' into patch-1 - Comment can be made more accurate - Merge branch 'openjdk:master' into patch-1 - Merge branch 'openjdk:master' into patch-1 - Merge branch 'openjdk:master' into patch-1 - Add more documentation - Replace with the actual new check - Little documentation doesn't hurt - Prepare for actual changes - ... and 7 more: https://git.openjdk.org/jdk/compare/c45090f7...5f97c5c7 ------------- Changes: - all: https://git.openjdk.org/jdk/pull/10882/files - new: https://git.openjdk.org/jdk/pull/10882/files/df46df9d..5f97c5c7 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=10882&range=05 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10882&range=04-05 Stats: 2029 lines in 70 files changed: 1414 ins; 247 del; 368 mod Patch: https://git.openjdk.org/jdk/pull/10882.diff Fetch: git fetch https://git.openjdk.org/jdk pull/10882/head:pull/10882 PR: https://git.openjdk.org/jdk/pull/10882 From jiefu at openjdk.org Tue Nov 1 07:28:19 2022 From: jiefu at openjdk.org (Jie Fu) Date: Tue, 1 Nov 2022 07:28:19 GMT Subject: RFR: 8295970: Add vector api sanity tests in tier1 [v2] In-Reply-To: <8uMsheGwjIBbgf1nJeYkCR4Pt_ddbnq4WVKMRPwS7C0=.c59b10eb-67e1-439c-936e-857c8430c520@github.com> References: <8uMsheGwjIBbgf1nJeYkCR4Pt_ddbnq4WVKMRPwS7C0=.c59b10eb-67e1-439c-936e-857c8430c520@github.com> Message-ID: On Fri, 28 Oct 2022 07:19:31 GMT, Jie Fu wrote: >> Hi all, >> >> As discussed here https://github.com/openjdk/jdk/pull/10807#pullrequestreview-1150314487 , it would be better to add the vector api tests in GHA. >> >> Thanks. >> Best regards, >> Jie > > Jie Fu 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 four additional commits since the last revision: > > - Add jdk_vector_sanity test group > - Merge branch 'master' into JDK-8295970 > - Revert changes in test.yml > - 8295970: Add jdk_vector tests in GHA Hi all, I changed the JBS title as `Add vector api sanity tests in tier1`. And the added `jdk_vector_sanity` tests can be run in less than 2min on my testing box. Any comments? Thanks. ------------- PR: https://git.openjdk.org/jdk/pull/10879 From ihse at openjdk.org Tue Nov 1 11:50:30 2022 From: ihse at openjdk.org (Magnus Ihse Bursie) Date: Tue, 1 Nov 2022 11:50:30 GMT Subject: RFR: 8295663 Rephrased introduction to testing [v2] In-Reply-To: <9TATzQxqQMn9tTHYFht7K5kBQMgP1rcHchmD-CDf8L4=.3a983daf-28ec-4d5a-bc12-912cbcdb3f23@github.com> References: <9TATzQxqQMn9tTHYFht7K5kBQMgP1rcHchmD-CDf8L4=.3a983daf-28ec-4d5a-bc12-912cbcdb3f23@github.com> Message-ID: On Wed, 26 Oct 2022 13:02:41 GMT, Ludvig Janiuk wrote: >> Old one made little sense for a new reader > > Ludvig Janiuk has updated the pull request incrementally with one additional commit since the last revision: > > reflow Marked as reviewed by ihse (Reviewer). ------------- PR: https://git.openjdk.org/jdk/pull/10763 From dfuchs at openjdk.org Tue Nov 1 12:21:13 2022 From: dfuchs at openjdk.org (Daniel Fuchs) Date: Tue, 1 Nov 2022 12:21:13 GMT Subject: RFR: 8295729: Add jcheck whitespace checking for properties files [v3] In-Reply-To: References: Message-ID: On Mon, 24 Oct 2022 19:21:07 GMT, Magnus Ihse Bursie wrote: >> Properties files is essentially source code. It should have the same whitespace checks as all other source code, so we don't get spurious trailing whitespace changes. >> >> With the new Skara jcheck, it is possible to increase the coverage of the whitespace checks (in the old mercurial version, this was more or less impossible). >> >> The only manual change is to `.jcheck/conf`. All other changes were made by running `find . -type f -iname "*.properties" | xargs gsed -i -e 's/[ \t]*$//'`. > > Magnus Ihse Bursie has updated the pull request incrementally with two additional commits since the last revision: > > - Revert "Remove check for .properties from jcheck" > > This reverts commit c91fdaa19dc06351598bd1c0614e1af3bfa08ae2. > - Change trailing space and tab in values to unicode encoding Changes to java (resp sun) .util.logging tests and javax.management tests look good to me. ------------- Marked as reviewed by dfuchs (Reviewer). PR: https://git.openjdk.org/jdk/pull/10792 From erikj at openjdk.org Tue Nov 1 12:44:33 2022 From: erikj at openjdk.org (Erik Joelsson) Date: Tue, 1 Nov 2022 12:44:33 GMT Subject: RFR: JDK-8295914: Add a header to generated HTML files in specs [v2] In-Reply-To: References: Message-ID: On Mon, 31 Oct 2022 23:23:44 GMT, Jonathan Gibbons wrote: >> Please review some updates to the parts of the build system related to building the documentation. > > Jonathan Gibbons 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 three additional commits since the last revision: > > - Merge remote-tracking branch 'upstream/master' into 8295914.spec-headers > - Address review feedback > - JDK-8295914: Add a header to generated HTML files in specs Marked as reviewed by erikj (Reviewer). ------------- PR: https://git.openjdk.org/jdk/pull/10891 From erikj at openjdk.org Tue Nov 1 12:47:29 2022 From: erikj at openjdk.org (Erik Joelsson) Date: Tue, 1 Nov 2022 12:47:29 GMT Subject: RFR: 8295970: Add vector api sanity tests in tier1 [v2] In-Reply-To: <8uMsheGwjIBbgf1nJeYkCR4Pt_ddbnq4WVKMRPwS7C0=.c59b10eb-67e1-439c-936e-857c8430c520@github.com> References: <8uMsheGwjIBbgf1nJeYkCR4Pt_ddbnq4WVKMRPwS7C0=.c59b10eb-67e1-439c-936e-857c8430c520@github.com> Message-ID: On Fri, 28 Oct 2022 07:19:31 GMT, Jie Fu wrote: >> Hi all, >> >> As discussed here https://github.com/openjdk/jdk/pull/10807#pullrequestreview-1150314487 , it would be better to add the vector api tests in GHA. >> >> Thanks. >> Best regards, >> Jie > > Jie Fu 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 four additional commits since the last revision: > > - Add jdk_vector_sanity test group > - Merge branch 'master' into JDK-8295970 > - Revert changes in test.yml > - 8295970: Add jdk_vector tests in GHA This looks good to me. The group tier1_part3 looks like it's currently the fastest of the 3, so it's the right one to add to. ------------- Marked as reviewed by erikj (Reviewer). PR: https://git.openjdk.org/jdk/pull/10879 From apavlyutkin at azul.com Tue Nov 1 07:48:38 2022 From: apavlyutkin at azul.com (Alexey Pavlyutkin) Date: Tue, 1 Nov 2022 07:48:38 +0000 Subject: [jdk8u-dev] RFR: 8197859: VS2017 Complains about UINTPTR_MAX definition in globalDefinitions_VisCPP.hpp Message-ID: Hello guys! Can I ask you to review https://github.com/openjdk/jdk8u-dev/pull/77 This is one of the seria of backports required to add support of MSVS2019 to jdk-8 and it blocks the further activities. Thank you Regards, Alex -------------- next part -------------- An HTML attachment was scrubbed... URL: From kvn at openjdk.org Tue Nov 1 16:15:24 2022 From: kvn at openjdk.org (Vladimir Kozlov) Date: Tue, 1 Nov 2022 16:15:24 GMT Subject: RFR: 8295970: Add vector api sanity tests in tier1 [v2] In-Reply-To: <8uMsheGwjIBbgf1nJeYkCR4Pt_ddbnq4WVKMRPwS7C0=.c59b10eb-67e1-439c-936e-857c8430c520@github.com> References: <8uMsheGwjIBbgf1nJeYkCR4Pt_ddbnq4WVKMRPwS7C0=.c59b10eb-67e1-439c-936e-857c8430c520@github.com> Message-ID: <4W4-SzCjMp5ML_70PyjaweJPxaBaBZuT728K3OS4sJc=.5d1b3fec-051d-4b12-bd30-b839f2368c5c@github.com> On Fri, 28 Oct 2022 07:19:31 GMT, Jie Fu wrote: >> Hi all, >> >> As discussed here https://github.com/openjdk/jdk/pull/10807#pullrequestreview-1150314487 , it would be better to add the vector api tests in GHA. >> >> Thanks. >> Best regards, >> Jie > > Jie Fu 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 four additional commits since the last revision: > > - Add jdk_vector_sanity test group > - Merge branch 'master' into JDK-8295970 > - Revert changes in test.yml > - 8295970: Add jdk_vector tests in GHA Good. I think we need review from @shipilev too since he had strong opinion about this. ------------- Marked as reviewed by kvn (Reviewer). PR: https://git.openjdk.org/jdk/pull/10879 From jjg at openjdk.org Tue Nov 1 18:42:27 2022 From: jjg at openjdk.org (Jonathan Gibbons) Date: Tue, 1 Nov 2022 18:42:27 GMT Subject: Integrated: JDK-8295914: Add a header to generated HTML files in specs In-Reply-To: References: Message-ID: On Thu, 27 Oct 2022 20:50:04 GMT, Jonathan Gibbons wrote: > Please review some updates to the parts of the build system related to building the documentation. This pull request has now been integrated. Changeset: d17bf51f Author: Jonathan Gibbons URL: https://git.openjdk.org/jdk/commit/d17bf51f91371072a758e775136af946192e771b Stats: 144 lines in 3 files changed: 139 ins; 1 del; 4 mod 8295914: Add a header to generated HTML files in specs Reviewed-by: erikj, iris ------------- PR: https://git.openjdk.org/jdk/pull/10891 From jiefu at openjdk.org Wed Nov 2 01:28:07 2022 From: jiefu at openjdk.org (Jie Fu) Date: Wed, 2 Nov 2022 01:28:07 GMT Subject: RFR: 8295970: Add vector api sanity tests in tier1 [v2] In-Reply-To: References: <8uMsheGwjIBbgf1nJeYkCR4Pt_ddbnq4WVKMRPwS7C0=.c59b10eb-67e1-439c-936e-857c8430c520@github.com> Message-ID: On Fri, 28 Oct 2022 13:41:28 GMT, Erik Joelsson wrote: >> Jie Fu 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 four additional commits since the last revision: >> >> - Add jdk_vector_sanity test group >> - Merge branch 'master' into JDK-8295970 >> - Revert changes in test.yml >> - 8295970: Add jdk_vector tests in GHA > > I think you need to add at least one other label than `build` to this now to make sure the right people can have a say in the change. Thanks @erikj79 and @vnkozlov for the review. So @shipilev , are you fine with this change? Thanks. ------------- PR: https://git.openjdk.org/jdk/pull/10879 From jwaters at openjdk.org Wed Nov 2 10:23:46 2022 From: jwaters at openjdk.org (Julian Waters) Date: Wed, 2 Nov 2022 10:23:46 GMT Subject: RFR: 8295990: Improve make handling of strip flags [v7] In-Reply-To: References: Message-ID: > Strip currently has its flags set by guessing based on the OS, it would be more appropriate to instead set them based on properly checking the strip binary instead of speculating what the correct flags are based on the compiler and OS. Julian Waters 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 18 additional commits since the last revision: - Merge branch 'openjdk:master' into patch-1 - Merge branch 'openjdk:master' into patch-1 - Merge branch 'openjdk:master' into patch-1 - Comment can be made more accurate - Merge branch 'openjdk:master' into patch-1 - Merge branch 'openjdk:master' into patch-1 - Merge branch 'openjdk:master' into patch-1 - Add more documentation - Replace with the actual new check - Little documentation doesn't hurt - ... and 8 more: https://git.openjdk.org/jdk/compare/d4b0fa17...25bb4a3c ------------- Changes: - all: https://git.openjdk.org/jdk/pull/10882/files - new: https://git.openjdk.org/jdk/pull/10882/files/5f97c5c7..25bb4a3c Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=10882&range=06 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10882&range=05-06 Stats: 10848 lines in 221 files changed: 8133 ins; 1765 del; 950 mod Patch: https://git.openjdk.org/jdk/pull/10882.diff Fetch: git fetch https://git.openjdk.org/jdk pull/10882/head:pull/10882 PR: https://git.openjdk.org/jdk/pull/10882 From jwaters at openjdk.org Wed Nov 2 10:26:37 2022 From: jwaters at openjdk.org (Julian Waters) Date: Wed, 2 Nov 2022 10:26:37 GMT Subject: RFR: 8295990: Improve make handling of strip flags [v5] In-Reply-To: References: Message-ID: On Mon, 31 Oct 2022 12:58:17 GMT, Erik Joelsson wrote: >> Julian Waters 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 16 additional commits since the last revision: >> >> - Merge branch 'openjdk:master' into patch-1 >> - Comment can be made more accurate >> - Merge branch 'openjdk:master' into patch-1 >> - Merge branch 'openjdk:master' into patch-1 >> - Merge branch 'openjdk:master' into patch-1 >> - Add more documentation >> - Replace with the actual new check >> - Little documentation doesn't hurt >> - Prepare for actual changes >> - Update flags-other.m4 >> - ... and 6 more: https://git.openjdk.org/jdk/compare/bbd65afa...df46df9d > > Marked as reviewed by erikj (Reviewer). @erikj79 Could i trouble you for a sponsor? Thanks! ------------- PR: https://git.openjdk.org/jdk/pull/10882 From jwaters at openjdk.org Wed Nov 2 11:55:34 2022 From: jwaters at openjdk.org (Julian Waters) Date: Wed, 2 Nov 2022 11:55:34 GMT Subject: RFR: 8296115: Allow for compiling the JDK with strict standards conformance [v3] In-Reply-To: References: Message-ID: > [JDK-8241499](https://bugs.openjdk.org/browse/JDK-8241499) proposes to set the -permissive- flag for the Microsoft Visual C++ compiler, to enforce strict standards conformance during compilation, making native code behave more strictly. While adding it to default builds is likely not practical given how much testing is required, as an option it can prove helpful in finding areas of native code that are not conformant to the standard. Instead of applying this to just one compiler, we can also include this for every compiler that has support for such a strict mode, which this change does. Julian Waters 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 five additional commits since the last revision: - Merge remote-tracking branch 'upstream/master' into permissive - Merge remote-tracking branch 'upstream/master' into permissive - Format - Keep docs up to date - New --enable-conforming-compilation option ------------- Changes: - all: https://git.openjdk.org/jdk/pull/10912/files - new: https://git.openjdk.org/jdk/pull/10912/files/7a527bc8..de41a935 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=10912&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10912&range=01-02 Stats: 11094 lines in 228 files changed: 8335 ins; 1792 del; 967 mod Patch: https://git.openjdk.org/jdk/pull/10912.diff Fetch: git fetch https://git.openjdk.org/jdk pull/10912/head:pull/10912 PR: https://git.openjdk.org/jdk/pull/10912 From jwaters at openjdk.org Wed Nov 2 11:56:47 2022 From: jwaters at openjdk.org (Julian Waters) Date: Wed, 2 Nov 2022 11:56:47 GMT Subject: RFR: 8295884: Implement IDE support for Eclipse [v13] In-Reply-To: References: Message-ID: > Eclipse is a popular and very well-known IDE in the world of Java development, utilized widely in many contexts, by beginners and experienced teams alike. Although a relatively lightweight IDE, it features surprisingly powerful indexing and code analysis capabilities, as well as useful tools, among which are make and Ant integration. While the tools it provides are not always as sophisticated as other IDEs (IntelliJ IDEA will likely come to mind as one such competitor), the simplicity of using it, as well as the reliability of this rugged IDE makes up greatly for the slightly less advanced tooling. Eclipse requires very little starting infrastructure in the workspace for all these features and indexing support as well, which makes it a good candidate for developing on the JDK. > > This enhancement adds 4 extra targets to the make system for generating a basic Eclipse Workspace that provides almost full indexing support for the JDK, with varying levels as desired, from a minimalistic option only including the JVM's sources, to generating a workspace with both Java and C/C++ natures included, which allows for using Eclipse's ability to quickly swap between Java and C/C++ mode to work on both native and Java sources at the same time. Cross Compiling support is available, and in its entirety the change touches very little of the existing make system, barring its own Makefile within the ide subdirectory. > > Indexing capabilities utilizing the enhancement: > java > escape Julian Waters 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 17 additional commits since the last revision: - Merge remote-tracking branch 'upstream/master' into eclipse - Merge remote-tracking branch 'upstream/master' into eclipse - Merge remote-tracking branch 'upstream/master' into eclipse - Include extra directories required by HotSpot in HotSpot only workspaces - SINGLE_DOLLAR is not required - Nuke accidental comment not meant to be added - Initial support for Language Settings Providers - Prevent Eclipse from automatically attaching binary parsers to workspaces including native code - Merge - Squash issue with commas - ... and 7 more: https://git.openjdk.org/jdk/compare/3e3e9c16...065cbea5 ------------- Changes: - all: https://git.openjdk.org/jdk/pull/10853/files - new: https://git.openjdk.org/jdk/pull/10853/files/db0e248e..065cbea5 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=10853&range=12 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10853&range=11-12 Stats: 11094 lines in 228 files changed: 8335 ins; 1792 del; 967 mod Patch: https://git.openjdk.org/jdk/pull/10853.diff Fetch: git fetch https://git.openjdk.org/jdk pull/10853/head:pull/10853 PR: https://git.openjdk.org/jdk/pull/10853 From jwaters at openjdk.org Wed Nov 2 11:56:52 2022 From: jwaters at openjdk.org (Julian Waters) Date: Wed, 2 Nov 2022 11:56:52 GMT Subject: RFR: 8296115: Allow for compiling the JDK with strict standards conformance [v2] In-Reply-To: References: Message-ID: On Tue, 1 Nov 2022 00:12:00 GMT, Julian Waters wrote: >> [JDK-8241499](https://bugs.openjdk.org/browse/JDK-8241499) proposes to set the -permissive- flag for the Microsoft Visual C++ compiler, to enforce strict standards conformance during compilation, making native code behave more strictly. While adding it to default builds is likely not practical given how much testing is required, as an option it can prove helpful in finding areas of native code that are not conformant to the standard. Instead of applying this to just one compiler, we can also include this for every compiler that has support for such a strict mode, which this change does. > > Julian Waters 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 four additional commits since the last revision: > > - Merge remote-tracking branch 'upstream/master' into permissive > - Format > - Keep docs up to date > - New --enable-conforming-compilation option Tagging @magicus as well, sorry for the noise ------------- PR: https://git.openjdk.org/jdk/pull/10912 From jwaters at openjdk.org Wed Nov 2 13:11:26 2022 From: jwaters at openjdk.org (Julian Waters) Date: Wed, 2 Nov 2022 13:11:26 GMT Subject: Integrated: 8295990: Improve make handling of strip flags In-Reply-To: References: Message-ID: On Thu, 27 Oct 2022 13:38:04 GMT, Julian Waters wrote: > Strip currently has its flags set by guessing based on the OS, it would be more appropriate to instead set them based on properly checking the strip binary instead of speculating what the correct flags are based on the compiler and OS. This pull request has now been integrated. Changeset: dac6ecc8 Author: Julian Waters Committer: Erik Joelsson URL: https://git.openjdk.org/jdk/commit/dac6ecc857c527e9df7af68389b46d452ac59ae0 Stats: 28 lines in 2 files changed: 20 ins; 2 del; 6 mod 8295990: Improve make handling of strip flags Reviewed-by: erikj ------------- PR: https://git.openjdk.org/jdk/pull/10882 From shade at openjdk.org Wed Nov 2 15:14:36 2022 From: shade at openjdk.org (Aleksey Shipilev) Date: Wed, 2 Nov 2022 15:14:36 GMT Subject: RFR: 8295970: Add vector api sanity tests in tier1 [v2] In-Reply-To: <8uMsheGwjIBbgf1nJeYkCR4Pt_ddbnq4WVKMRPwS7C0=.c59b10eb-67e1-439c-936e-857c8430c520@github.com> References: <8uMsheGwjIBbgf1nJeYkCR4Pt_ddbnq4WVKMRPwS7C0=.c59b10eb-67e1-439c-936e-857c8430c520@github.com> Message-ID: On Fri, 28 Oct 2022 07:19:31 GMT, Jie Fu wrote: >> Hi all, >> >> As discussed here https://github.com/openjdk/jdk/pull/10807#pullrequestreview-1150314487 , it would be better to add the vector api tests in GHA. >> >> Thanks. >> Best regards, >> Jie > > Jie Fu 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 four additional commits since the last revision: > > - Add jdk_vector_sanity test group > - Merge branch 'master' into JDK-8295970 > - Revert changes in test.yml > - 8295970: Add jdk_vector tests in GHA I think the usual style for these files is to maintain original order/sorting. test/jdk/TEST.groups line 44: > 42: :jdk_vector_sanity \ > 43: :jdk_svc_sanity \ > 44: :jdk_foreign \ Suggestion: :jdk_math \ :jdk_svc_sanity \ :jdk_foreign \ :jdk_vector_sanity \ test/jdk/TEST.groups line 80: > 78: :jdk_svc \ > 79: -:jdk_vector_sanity \ > 80: -:jdk_svc_sanity \ Suggestion: :jdk_svc \ -:jdk_svc_sanity \ -:jdk_vector_sanity \ test/jdk/TEST.groups line 367: > 365: > 366: jdk_vector_sanity = \ > 367: jdk/incubator/vector/AddTest.java \ These should probably be sorted too. ------------- PR: https://git.openjdk.org/jdk/pull/10879 From ihse at openjdk.org Wed Nov 2 15:34:52 2022 From: ihse at openjdk.org (Magnus Ihse Bursie) Date: Wed, 2 Nov 2022 15:34:52 GMT Subject: RFR: 8296115: Allow for compiling the JDK with strict standards conformance [v3] In-Reply-To: References: Message-ID: On Wed, 2 Nov 2022 11:55:34 GMT, Julian Waters wrote: >> [JDK-8241499](https://bugs.openjdk.org/browse/JDK-8241499) proposes to set the -permissive- flag for the Microsoft Visual C++ compiler, to enforce strict standards conformance during compilation, making native code behave more strictly. While adding it to default builds is likely not practical given how much testing is required, as an option it can prove helpful in finding areas of native code that are not conformant to the standard. Instead of applying this to just one compiler, we can also include this for every compiler that has support for such a strict mode, which this change does. > > Julian Waters 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 five additional commits since the last revision: > > - Merge remote-tracking branch 'upstream/master' into permissive > - Merge remote-tracking branch 'upstream/master' into permissive > - Format > - Keep docs up to date > - New --enable-conforming-compilation option Looks good to me. Goes the GHA test suite pass with this flag? If so, I think it would make sense to turn it on for our GHA builds. That will help raise the bar for future code coming in, and give this option some real testing. make/autoconf/spec.gmk.in line 333: > 331: ENABLE_HEADLESS_ONLY := @ENABLE_HEADLESS_ONLY@ > 332: > 333: ENABLE_CONFORMING_COMPILATION := @ENABLE_CONFORMING_COMPILATION@ You don't need to export this in spec.gmk, since it is only checked in the configure script. We try hard to only export here what is strictly necessary for use by makefiles. If you will ever need it in the future, please add it as an export here at that time. ------------- Marked as reviewed by ihse (Reviewer). PR: https://git.openjdk.org/jdk/pull/10912 From ihse at openjdk.org Wed Nov 2 15:38:16 2022 From: ihse at openjdk.org (Magnus Ihse Bursie) Date: Wed, 2 Nov 2022 15:38:16 GMT Subject: RFR: 8295812: Skip the "half float" support in LittleCMS during the build In-Reply-To: References: Message-ID: On Sun, 23 Oct 2022 20:10:35 GMT, Sergey Bylokhov wrote: > The Java2d do not use "half" float in the image layouts, so we can disable it in the LittleCMS library during the build. It is possible to do using [this](https://github.com/openjdk/jdk/blob/master/src/java.desktop/share/native/liblcms/lcms2.h#L85) public option: > > > // Uncomment to get rid of the tables for "half" float support > // #define CMS_NO_HALF_SUPPORT 1 > > > This change cuts not only the unused by java2d functions but also [such](https://github.com/openjdk/jdk/blob/739769c8fc4b496f08a92225a12d07414537b6c0/src/java.desktop/share/native/liblcms/cmshalf.c#L63) tables. > > The size of liblcms decreased by 20 kB on Linux(536024 vs 515152) and 15 kB on windows(246784 vs 231424). On macOS the win is only 300 bytes. Marked as reviewed by ihse (Reviewer). Nicely spotted! ------------- PR: https://git.openjdk.org/jdk/pull/10830 From ihse at openjdk.org Wed Nov 2 15:39:41 2022 From: ihse at openjdk.org (Magnus Ihse Bursie) Date: Wed, 2 Nov 2022 15:39:41 GMT Subject: RFR: 8294591: Fix cast-function-type warning in TemplateTable [v4] In-Reply-To: <2jHTulm9tMRZEq_tFq7UMUJjWwIS7TIoOkfxjVsseG8=.e69ac9a2-cf39-4972-818a-77cdf0ed93d6@github.com> References: <2jHTulm9tMRZEq_tFq7UMUJjWwIS7TIoOkfxjVsseG8=.e69ac9a2-cf39-4972-818a-77cdf0ed93d6@github.com> Message-ID: <-lSQrZXaxtGjPL-LuHHvroS8HOTvVdWgAx4J2yIAm60=.e932a8fd-8218-4068-96f0-3df02cf1d508@github.com> On Mon, 17 Oct 2022 18:08:56 GMT, Aleksey Shipilev wrote: >> After [JDK-8294314](https://bugs.openjdk.org/browse/JDK-8294314), we would have `templateTable.cpp` excluded with cast-function-type warning. The underlying cause for it is casting functions for `ldc` bytecodes, which take `bool`-typed handlers: > > Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision: > > Fix build failures Build changes are trivially good. I have not looked at hotspot source code changes. ------------- Marked as reviewed by ihse (Reviewer). PR: https://git.openjdk.org/jdk/pull/10493 From coleenp at openjdk.org Wed Nov 2 16:16:26 2022 From: coleenp at openjdk.org (Coleen Phillimore) Date: Wed, 2 Nov 2022 16:16:26 GMT Subject: RFR: 8294591: Fix cast-function-type warning in TemplateTable [v4] In-Reply-To: <2jHTulm9tMRZEq_tFq7UMUJjWwIS7TIoOkfxjVsseG8=.e69ac9a2-cf39-4972-818a-77cdf0ed93d6@github.com> References: <2jHTulm9tMRZEq_tFq7UMUJjWwIS7TIoOkfxjVsseG8=.e69ac9a2-cf39-4972-818a-77cdf0ed93d6@github.com> Message-ID: On Mon, 17 Oct 2022 18:08:56 GMT, Aleksey Shipilev wrote: >> After [JDK-8294314](https://bugs.openjdk.org/browse/JDK-8294314), we would have `templateTable.cpp` excluded with cast-function-type warning. The underlying cause for it is casting functions for `ldc` bytecodes, which take `bool`-typed handlers: > > Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision: > > Fix build failures Seems ok. ------------- Marked as reviewed by coleenp (Reviewer). PR: https://git.openjdk.org/jdk/pull/10493 From jwaters at openjdk.org Wed Nov 2 16:45:35 2022 From: jwaters at openjdk.org (Julian Waters) Date: Wed, 2 Nov 2022 16:45:35 GMT Subject: RFR: 8296115: Allow for compiling the JDK with strict standards conformance [v4] In-Reply-To: References: Message-ID: > [JDK-8241499](https://bugs.openjdk.org/browse/JDK-8241499) proposes to set the -permissive- flag for the Microsoft Visual C++ compiler, to enforce strict standards conformance during compilation, making native code behave more strictly. While adding it to default builds is likely not practical given how much testing is required, as an option it can prove helpful in finding areas of native code that are not conformant to the standard. Instead of applying this to just one compiler, we can also include this for every compiler that has support for such a strict mode, which this change does. Julian Waters 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 seven additional commits since the last revision: - Export not required - Merge remote-tracking branch 'upstream/master' into permissive - Merge remote-tracking branch 'upstream/master' into permissive - Merge remote-tracking branch 'upstream/master' into permissive - Format - Keep docs up to date - New --enable-conforming-compilation option ------------- Changes: - all: https://git.openjdk.org/jdk/pull/10912/files - new: https://git.openjdk.org/jdk/pull/10912/files/de41a935..1dd9765c Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=10912&range=03 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10912&range=02-03 Stats: 1572 lines in 69 files changed: 254 ins; 1196 del; 122 mod Patch: https://git.openjdk.org/jdk/pull/10912.diff Fetch: git fetch https://git.openjdk.org/jdk pull/10912/head:pull/10912 PR: https://git.openjdk.org/jdk/pull/10912 From jwaters at openjdk.org Wed Nov 2 16:45:37 2022 From: jwaters at openjdk.org (Julian Waters) Date: Wed, 2 Nov 2022 16:45:37 GMT Subject: RFR: 8296115: Allow for compiling the JDK with strict standards conformance [v3] In-Reply-To: References: Message-ID: On Wed, 2 Nov 2022 15:31:15 GMT, Magnus Ihse Bursie wrote: >> Julian Waters 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 five additional commits since the last revision: >> >> - Merge remote-tracking branch 'upstream/master' into permissive >> - Merge remote-tracking branch 'upstream/master' into permissive >> - Format >> - Keep docs up to date >> - New --enable-conforming-compilation option > > make/autoconf/spec.gmk.in line 333: > >> 331: ENABLE_HEADLESS_ONLY := @ENABLE_HEADLESS_ONLY@ >> 332: >> 333: ENABLE_CONFORMING_COMPILATION := @ENABLE_CONFORMING_COMPILATION@ > > You don't need to export this in spec.gmk, since it is only checked in the configure script. We try hard to only export here what is strictly necessary for use by makefiles. If you will ever need it in the future, please add it as an export here at that time. Alright, will do ------------- PR: https://git.openjdk.org/jdk/pull/10912 From erikj at openjdk.org Wed Nov 2 17:04:18 2022 From: erikj at openjdk.org (Erik Joelsson) Date: Wed, 2 Nov 2022 17:04:18 GMT Subject: RFR: 8296115: Allow for compiling the JDK with strict standards conformance [v4] In-Reply-To: References: Message-ID: On Wed, 2 Nov 2022 16:45:35 GMT, Julian Waters wrote: >> [JDK-8241499](https://bugs.openjdk.org/browse/JDK-8241499) proposes to set the -permissive- flag for the Microsoft Visual C++ compiler, to enforce strict standards conformance during compilation, making native code behave more strictly. While adding it to default builds is likely not practical given how much testing is required, as an option it can prove helpful in finding areas of native code that are not conformant to the standard. Instead of applying this to just one compiler, we can also include this for every compiler that has support for such a strict mode, which this change does. > > Julian Waters 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 seven additional commits since the last revision: > > - Export not required > - Merge remote-tracking branch 'upstream/master' into permissive > - Merge remote-tracking branch 'upstream/master' into permissive > - Merge remote-tracking branch 'upstream/master' into permissive > - Format > - Keep docs up to date > - New --enable-conforming-compilation option Marked as reviewed by erikj (Reviewer). ------------- PR: https://git.openjdk.org/jdk/pull/10912 From jwaters at openjdk.org Wed Nov 2 17:04:19 2022 From: jwaters at openjdk.org (Julian Waters) Date: Wed, 2 Nov 2022 17:04:19 GMT Subject: RFR: 8296115: Allow for compiling the JDK with strict standards conformance [v3] In-Reply-To: References: Message-ID: On Wed, 2 Nov 2022 15:32:51 GMT, Magnus Ihse Bursie wrote: > Goes the GHA test suite pass with this flag? If so, I think it would make sense to turn it on for our GHA builds. That will help raise the bar for future code coming in, and give this option some real testing. Given my experience with these errors I doubt it will work off the bat with some of the particularly strict compilers, such as gcc for instance. I'll leave the decision to enable this flag to someone else for now ------------- PR: https://git.openjdk.org/jdk/pull/10912 From jwaters at openjdk.org Wed Nov 2 17:05:59 2022 From: jwaters at openjdk.org (Julian Waters) Date: Wed, 2 Nov 2022 17:05:59 GMT Subject: Integrated: 8296115: Allow for compiling the JDK with strict standards conformance In-Reply-To: References: Message-ID: On Mon, 31 Oct 2022 03:22:32 GMT, Julian Waters wrote: > [JDK-8241499](https://bugs.openjdk.org/browse/JDK-8241499) proposes to set the -permissive- flag for the Microsoft Visual C++ compiler, to enforce strict standards conformance during compilation, making native code behave more strictly. While adding it to default builds is likely not practical given how much testing is required, as an option it can prove helpful in finding areas of native code that are not conformant to the standard. Instead of applying this to just one compiler, we can also include this for every compiler that has support for such a strict mode, which this change does. This pull request has now been integrated. Changeset: a124d8e0 Author: Julian Waters Committer: Erik Joelsson URL: https://git.openjdk.org/jdk/commit/a124d8e0479f4c59d27c69c6b580542b394fd02f Stats: 41 lines in 4 files changed: 39 ins; 0 del; 2 mod 8296115: Allow for compiling the JDK with strict standards conformance 8241499: Enable new "permissive-" for standard C++ compliance on Visual Studio if possible Reviewed-by: erikj, ihse ------------- PR: https://git.openjdk.org/jdk/pull/10912 From jjg at openjdk.org Wed Nov 2 19:14:20 2022 From: jjg at openjdk.org (Jonathan Gibbons) Date: Wed, 2 Nov 2022 19:14:20 GMT Subject: RFR: JDK-8296188: Update style and header in JDWP Protocol spec and JVMTI spec Message-ID: Please review a simple makefile-only update to inject the standard stylesheet and navigation header into the two JDK specifications that are generated as HTML files in the `gensrc` directory, to bring these docs in line with the rest of the specs documents. Generated docs at: * http://cr.openjdk.java.net/~jjg/8296188/docs.00/specs/jdwp/jdwp-protocol.html * http://cr.openjdk.java.net/~jjg/8296188/docs.00/specs/jvmti.html The JVM TI spec has a list of links at the beginning, in bold. This is not a CSS issue ... it is in the HTML, generated by XSL scripts. Ideally, the list should be in a separate `