RFR: 8254072: AArch64: Get rid of --disable-warnings-as-errors on Windows+ARM64 build [v4]
Bernhard Urban-Forster
burban at openjdk.java.net
Mon Nov 2 17:07:54 UTC 2020
On Mon, 2 Nov 2020 16:16:25 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
>> @magicus I did test the initial version of this PR on linux+arm64, but not the latest iteration. sorry about that
>>
>> What is the policy here? Submit a revert right away or investigate a fix?
>
> @lewurm Open a new JBS issue with the bug. If you can find a fix in a short amount of time (which I would believe should be possible; probably just need a proper cast) it's acceptable to fix it directly. What amounts to a "short amount of time" is left to reasonable judgement; minutes to hours are okay, days are not.
>
> Otherwise, create an anti-delta (revert changeset) to back out your changes, and open yet another JBS issue for re-implementing them correctly.
>
> In this case, an alternative short-term fix could also be to remove the assert instead of backing out the entire patch.
https://github.com/openjdk/jdk/pull/1013
-------------
PR: https://git.openjdk.java.net/jdk/pull/530
More information about the hotspot-dev
mailing list