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

Christoph Langer clanger at openjdk.org
Tue Aug 16 19:20:27 UTC 2022


On Tue, 16 Aug 2022 16:48:56 GMT, Salander <duke 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.

I think it would be a good start.

> > 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.

If you aren't in the OpenJDK Census, you won't get an account, unless you have made some 3 commits to the OpenJDK. However, I can create a JBS issue for you.

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

Great. And we already see build failures 😉

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

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


More information about the jdk-updates-dev mailing list