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