RFR: JDK-8300069: Left shift of negative value in share/adlc/dict2.cpp [v3]

Kim Barrett kbarrett at openjdk.org
Sun Jan 15 02:26:14 UTC 2023


On Fri, 13 Jan 2023 15:57:22 GMT, Justin King <jcking at openjdk.org> wrote:

> Okay, I quickly looked at the C++23 draft standard and couldn't find much regarding negative integral on the left side and left shifting. So let's disable this check, as I agree it's being a bit to pedantic.

C++20 defines signed integers to be two's complement.
https://www.open-std.org/jtc1/sc22/wg21/docs/papers/2018/p1236r1.html

Left-shift of negative values has been discussed before: https://bugs.openjdk.org/browse/JDK-8240259
"Disable -Wshift-negative-value warnings"

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

PR: https://git.openjdk.org/jdk/pull/11977



More information about the build-dev mailing list