aboutsummaryrefslogtreecommitdiff
path: root/src/check_expr.cpp
diff options
context:
space:
mode:
authorgingerBill <gingerBill@users.noreply.github.com>2025-08-10 18:29:08 +0100
committergingerBill <gingerBill@users.noreply.github.com>2025-08-10 18:29:08 +0100
commit8df69c95c3163562b6caf6c55651363c17c3f478 (patch)
treedf65019c482eb0bbf6d090475809f448147a5251 /src/check_expr.cpp
parentb6e0b4848116697fa86a46c0b06a5ceb3654d5c6 (diff)
Add `-integer-division-by-zero:all-bits`
Diffstat (limited to 'src/check_expr.cpp')
-rw-r--r--src/check_expr.cpp31
1 files changed, 25 insertions, 6 deletions
diff --git a/src/check_expr.cpp b/src/check_expr.cpp
index e0b6408e3..7020b4f4b 100644
--- a/src/check_expr.cpp
+++ b/src/check_expr.cpp
@@ -4377,12 +4377,23 @@ gb_internal void check_binary_expr(CheckerContext *c, Operand *x, Ast *node, Typ
b.kind == ExactValue_Integer && big_int_is_zero(&b.value_integer) &&
(op.kind == Token_QuoEq || op.kind == Token_Mod || op.kind == Token_ModMod)) {
if (op.kind == Token_QuoEq) {
- if (zero_behaviour == IntegerDivisionByZero_Zero) {
+ switch (zero_behaviour) {
+ case IntegerDivisionByZero_Zero:
// x/0 == 0
x->value = b;
- } else {
+ break;
+ case IntegerDivisionByZero_Self:
// x/0 == x
x->value = a;
+ break;
+ case IntegerDivisionByZero_AllBits:
+ // x/0 == 0b111...111
+ if (is_type_untyped(x->type)) {
+ x->value = exact_value_i64(-1);
+ } else {
+ x->value = exact_unary_operator_value(Token_Xor, b, cast(i32)(8*type_size_of(x->type)), is_type_unsigned(x->type));
+ }
+ break;
}
} else {
/*
@@ -4391,16 +4402,21 @@ gb_internal void check_binary_expr(CheckerContext *c, Operand *x, Ast *node, Typ
truncated: r = a - b*trunc(a/b)
floored: r = a - b*floor(a/b)
- IFF a/0 == 0, then (a%0 == a) or (a%%0 == a)
- IFF a/0 == a, then (a%0 == 0) or (a%%0 == 0)
+ IFF a/0 == 0, then (a%0 == a) or (a%%0 == a)
+ IFF a/0 == a, then (a%0 == 0) or (a%%0 == 0)
+ IFF a/0 == 0b111..., then (a%0 == a) or (a%%0 == a)
*/
- if (zero_behaviour == IntegerDivisionByZero_Zero) {
+ switch (zero_behaviour) {
+ case IntegerDivisionByZero_Zero:
+ case IntegerDivisionByZero_AllBits:
// x%0 == x
x->value = a;
- } else {
+ break;
+ case IntegerDivisionByZero_Self:
// x%0 == 0
x->value = b;
+ break;
}
}
} else {
@@ -9670,6 +9686,9 @@ gb_internal IntegerDivisionByZeroKind check_for_integer_division_by_zero(Checker
if ((flags & OptInFeatureFlag_IntegerDivisionByZero_Self) != 0) {
return IntegerDivisionByZero_Self;
}
+ if ((flags & OptInFeatureFlag_IntegerDivisionByZero_AllBits) != 0) {
+ return IntegerDivisionByZero_AllBits;
+ }
return build_context.integer_division_by_zero_behaviour;
}