aboutsummaryrefslogtreecommitdiff
path: root/src/check_builtin.cpp
diff options
context:
space:
mode:
authorFranz <franz-hoeltermann@web.de>2026-01-14 00:43:06 +0100
committerFranz <franz-hoeltermann@web.de>2026-01-14 00:43:06 +0100
commitc64fad8ef7ab196854112c4a65992b11510fde90 (patch)
tree92294035036438bdb802f84cce72fcd8151a7a4b /src/check_builtin.cpp
parent9d72025a0b6f7b94682662316b892ece865ad8aa (diff)
Add warning for `size_of(&x)` expressions
Diffstat (limited to 'src/check_builtin.cpp')
-rw-r--r--src/check_builtin.cpp10
1 files changed, 10 insertions, 0 deletions
diff --git a/src/check_builtin.cpp b/src/check_builtin.cpp
index 1b3e6912c..7b31314f8 100644
--- a/src/check_builtin.cpp
+++ b/src/check_builtin.cpp
@@ -2695,6 +2695,16 @@ gb_internal bool check_builtin_procedure(CheckerContext *c, Operand *operand, As
case BuiltinProc_size_of: {
// size_of :: proc(Type or expr) -> untyped int
+ if (ce->args[0]->kind == Ast_UnaryExpr) {
+ ast_node(arg, UnaryExpr, ce->args[0]);
+ if (arg->op.kind == Token_And) {
+ ERROR_BLOCK();
+
+ warning(ce->args[0], "'size_of(&x)' returns the size of a pointer, not the size of x");
+ error_line("\tSuggestion: Use 'size_of(rawptr)' if you want the size of the pointer");
+ }
+ }
+
Operand o = {};
check_expr_or_type(c, &o, ce->args[0]);
if (o.mode == Addressing_Invalid) {