[jdk11u-dev] RFR: Allow building with Clang 13

Salander duke at openjdk.org
Tue Aug 16 16:51:36 UTC 2022


On Tue, 16 Aug 2022 13:34:33 GMT, Christoph Langer <clanger at openjdk.org> wrote:

> that's quite a big patch. Can this be made smaller - if the purpose is only to make clang13 work?

Renaming the `markOopDesc` class to `markOop` could be reverted which would reduce the diff down to ~58 changed files. Beyond that I'm not sure which changes could be dropped while still fixing the issue.

> Furthermore, as this is no full backport of JDK-8229258, I suggest to change the title into just "Allow building with Clang 13" for the time being. I think, if this is ever going to be reviewed and accepted, we'll need to open a specific JBS bug for this, targeting JDK 11.

I'll rename it. Not sure if I have the ability to create a JBS account, I did the password reset but didn't receive an email.

> I'm also not sure where support for clang13 is needed in the OpenJDK so far. Usually, linux versions are built with gcc. clang is only used on MacOS and AIX but for these OS versions we're not yet at clang version 13, I believe.

A few Linux distributions have switched to building with Clang because the end result is significantly smaller package sizes. FreeBSD also builds with clang and had to pin to clang12. 

> Another hint/requirement: Please enable GitHub Actions on your repository so as to have some basic tier1 acceptance testing for the PR.
Will do!

-------------

PR: https://git.openjdk.org/jdk11u-dev/pull/1284


More information about the jdk-updates-dev mailing list