RFR: JDK-8299554: Remove EnableIf from metaprogramming/enableIf.hpp
Kim Barrett
kbarrett at openjdk.org
Tue Jan 3 21:40:50 UTC 2023
On Tue, 3 Jan 2023 19:42:07 GMT, Justin King <jcking at openjdk.org> wrote:
> @kimbarrett Cleanup EnableIf, but leave ENABLE_IF and ENABLE_IF_SDEFN as discussed. Also switched to `std::enable_if_t`.
My suggestion here: https://github.com/openjdk/jdk/pull/11794#pullrequestreview-1231289245
was to leave this one alone. I think (pretty nearly) all current uses of EnableIf would be better handled by refactoring
to take advantage of C++17 if-constexpr. I think there's little point to this intermediate change if C++17 support is
coming (which I'm hopeful that it is).
-------------
PR: https://git.openjdk.org/jdk/pull/11835
More information about the hotspot-dev
mailing list