RFR: 8183552: Move align functions to align.hpp
Thomas Stüfe
thomas.stuefe at gmail.com
Wed Jul 5 06:10:45 UTC 2017
Hi Stefan,
this looks good. This change (and all the preceding ones) are nice cleanups!
Kind Regards, Thomas
On Wed, Jul 5, 2017 at 12:25 AM, Stefan Karlsson <stefan.karlsson at oracle.com
> wrote:
> Hi all,
>
> Please review this patch to move the align functions to align.hpp.
>
> http://cr.openjdk.java.net/~stefank/8183552/webrev.00
> https://bugs.openjdk.java.net/browse/JDK-8183552
>
> The patch mainly moves the functions and adding #include
> "utilities/align.hpp" lines, but it also moves ScaleForWordSize to
> globals.hpp to prevent a cyclic dependency between align.hpp and
> globalDefinitions.hpp.
>
> Thanks,
> StefanK
>
More information about the hotspot-dev
mailing list