[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