RFR: 8247910: Improve alignment and power-of-2 utilities using C++14

Kim Barrett kbarrett at openjdk.java.net
Fri Sep 11 08:50:54 UTC 2020


Please review this change which updates the alignment and power-of-2
utilities (utilities/align.hpp and utilities/powerOfTwo.hpp) to use
C++14 features.

Where possible, the alignment functions are now constexpr.  This
eliminates the need for alternate macros that needed to be used in
constexpr contexts.

Use <type_traits> and <limits> rather than HotSpot workalikes.
We no longer need max_value<T>(), as the problematic platform for
std::numeric_limits<T>::max() was Solaris.

Testing: tier1

-------------

Commit messages:
 - Update alignment and power-of-2 functions for C++14

Changes: https://git.openjdk.java.net/jdk/pull/126/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=126&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8247910
  Stats: 291 lines in 12 files changed: 129 ins; 90 del; 72 mod
  Patch: https://git.openjdk.java.net/jdk/pull/126.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/126/head:pull/126

PR: https://git.openjdk.java.net/jdk/pull/126


More information about the hotspot-dev mailing list