diff options
| author | xackus <14938807+xackus@users.noreply.github.com> | 2020-07-05 17:58:21 +0200 |
|---|---|---|
| committer | xackus <14938807+xackus@users.noreply.github.com> | 2020-07-05 17:58:21 +0200 |
| commit | 51f8c306d9fa32c849b26c2d95a45901a02f448e (patch) | |
| tree | 80a09b965eaa0ff2849f3eafc7405eb9af3146ff /src/codegen.cpp | |
| parent | 0ae1157e4553d6f54e0d489daebb006c402e0f63 (diff) | |
| download | zig-51f8c306d9fa32c849b26c2d95a45901a02f448e.tar.gz zig-51f8c306d9fa32c849b26c2d95a45901a02f448e.zip | |
stage1: add missing runtime safety for @intCast unsigned -> signed of same bit count
Diffstat (limited to 'src/codegen.cpp')
| -rw-r--r-- | src/codegen.cpp | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/src/codegen.cpp b/src/codegen.cpp index 18ef7d9182..2f72861bc2 100644 --- a/src/codegen.cpp +++ b/src/codegen.cpp @@ -1535,9 +1535,11 @@ static LLVMValueRef gen_widen_or_shorten(CodeGen *g, bool want_runtime_safety, Z zig_unreachable(); } - if (actual_type->id == ZigTypeIdInt && - !wanted_type->data.integral.is_signed && actual_type->data.integral.is_signed && - want_runtime_safety) + if (actual_type->id == ZigTypeIdInt && want_runtime_safety && ( + // negative to unsigned + (!wanted_type->data.integral.is_signed && actual_type->data.integral.is_signed) || + // unsigned would become negative + (wanted_type->data.integral.is_signed && !actual_type->data.integral.is_signed && actual_bits == wanted_bits))) { LLVMValueRef zero = LLVMConstNull(get_llvm_type(g, actual_type)); LLVMValueRef ok_bit = LLVMBuildICmp(g->builder, LLVMIntSGE, expr_val, zero, ""); @@ -1547,7 +1549,7 @@ static LLVMValueRef gen_widen_or_shorten(CodeGen *g, bool want_runtime_safety, Z LLVMBuildCondBr(g->builder, ok_bit, ok_block, fail_block); LLVMPositionBuilderAtEnd(g->builder, fail_block); - gen_safety_crash(g, PanicMsgIdCastNegativeToUnsigned); + gen_safety_crash(g, actual_type->data.integral.is_signed ? PanicMsgIdCastNegativeToUnsigned : PanicMsgIdCastTruncatedData); LLVMPositionBuilderAtEnd(g->builder, ok_block); } |
