RFR and workflow question

Vladimir Kozlov vladimir.kozlov at oracle.com
Wed Jan 20 16:46:48 UTC 2016


Use 8147475 for commit message - your change can be related (relocation info could be affected by padding).
But I don't see how your change can fix 8146608 - it is patching return PC which is SP relative and nothing to do with 
padding in prolog.

Changes looks fine. Please, integrate it first since it affect all hotspot repos.

Thanks,
Vladimir

On 1/20/16 7:28 AM, Roland Schatz wrote:
> Hi!
>
> Please review this small bugfix:
> http://cr.openjdk.java.net/~rschatz/JDK-8147475/webrev.00/
>
> This is on top of another webrev[1]. It has no semantic dependency on it, just a random source-level conflict. I can
> rebase them if we want to integrate this one first.
>
>
> This is missing a commit message, and I'm not sure what to use here.
>
> There are two issues that I *think* this will solve, but I can't be sure since I haven't managed to reproduce them:
> https://bugs.openjdk.java.net/browse/JDK-8147475
> https://bugs.openjdk.java.net/browse/JDK-8146608
>
> Do I just use one of the above issues randomly for the commit message, and close the other as duplicate?
>
>
> Thanks,
> Roland
>
> [1] http://cr.openjdk.java.net/~rschatz/JDK-8147599/webrev.00/


More information about the hotspot-compiler-dev mailing list