Why does sizeof(x++) not increment x?
Asked Answered
B

10

530

Here is the code compiled in Dev-C++ on Windows:

#include <stdio.h>

int main() {
    int x = 5;
    printf("%d and ", sizeof(x++)); // note 1
    printf("%d\n", x); // note 2
    return 0;
}

I expect x to be 6 after executing note 1. However, the output is:

4 and 5

Why does x not increment after note 1?

Bevel answered 22/11, 2011 at 11:7 Comment(3)
I'd note that DevC++ uses a very old outdated compiler, you may want to upgrade to a newer IDE, e.g. Codeblocks Eclipse or Visual StudioReconstruction
printf("%d and ", sizeof(x++)); // note 1 causes UB, why do you expect any meaningful output? Please read the printf() manpage or the C standard sections about printf()/ fprintf().Halftruth
Why does a C compiler have "++" in its name? That's just confusing.Formicary
T
563

From the C99 Standard (the emphasis is mine)

6.5.3.4/2

The sizeof operator yields the size (in bytes) of its operand, which may be an expression or the parenthesized name of a type. The size is determined from the type of the operand. The result is an integer. If the type of the operand is a variable length array type, the operand is evaluated; otherwise, the operand is not evaluated and the result is an integer constant.

Toxinantitoxin answered 22/11, 2011 at 11:11 Comment(7)
what do you mean by variable length array type? That does mean the operand is an array? The code in this case is not an array. Can you clear things up for me?Bevel
A variable length array is an array declared with the size being a value unknown during compilation, for instance if you read N from stdin and make int array[N]. This is one of C99 features, unavailable in C++.Metre
@LegendofCage, in particular this would mean that in something like sizeof(int[++x]) (really, really a bad idea, anyhow) the ++ could be evaluated.Labefaction
@Joe Wreschnig: That's plain incorrect. int[++x] has VLA type int[n] (for some value of n). Why would you say it is int???Culprit
This is not relevant for the code he provides, his code causes UB and that is what should be addressed.Halftruth
@Halftruth -- I disagree, this answer exactly addresses the question raised by OP. The point about UB IMO is irrelevant to the actual question in the post.Evesham
This behavior is actually quite nice because it means something like sizeof(*null_ptr) won't segfault when you're trying to find the size of the type the pointer is pointing to.Bimestrial
C
196

sizeof is a compile-time operator, so at the time of compilation sizeof and its operand get replaced by the result value. The operand is not evaluated (except when it is a variable length array) at all; only the type of the result matters.

short func(short x) {  // this function never gets called !!
   printf("%d", x);    // this print never happens
   return x;
}

int main() {
   printf("%d", sizeof(func(3))); // all that matters to sizeof is the 
                                  // return type of the function.
   return 0;
}

Output:

2

as short occupies 2 bytes on my machine.

Changing the return type of the function to double:

double func(short x) {
// rest all same

will give 8 as output.

Christelchristen answered 22/11, 2011 at 11:10 Comment(7)
Only sometimes - it's compile time if possible.Pyroclastic
There's a nice benefit to compile-time resolution of the sizeof() operator when working with strings. If you have a string that is initialized as a quoted string, instead of using strlen(), where the character array comprising the string has to be scanned for the null-terminator at run time, sizeof(quoted_string) is known at compile time, and therefore at run-time. It's a small thing, but if you use the quoted string in a loop millions and millions of times, it makes a significant difference in performance.Prattle
If you really use it millions and millions of times in a loop, wouldn't it be much more sensible to factor the length calculation out of the loop? I surely hope you don't have millions and millions of different hardcoded constants in your code. :-oHell
@Veky: Refactoring the length calculation out of the loop would still require tying up some kind of storage (likely a register) to hold it, which may result in an otherwise-avoidable register spill, and also require generating code to compute it. Simply having the compiler generate a constant is better in essentially every way imaginable.Kozloski
I really think that whichever such trivial code transformation we can do, compiler can do too. But maybe I'm mistaken.Hell
Your code causes UB just like the ones from the OP: printf("%d", sizeof(func(3))); This lines causes UB. When you enable compiler warnings it is likely that the compiler warns you about this line.Halftruth
Re "sizeof is a compile-time operator", No, Example: printf( "%zu\n", sizeof( int[ printf("!") ] ) ); prints something like !4Deidradeidre
B
51

sizeof(foo) tries really hard to discover the size of an expression at compile time:

6.5.3.4:

The sizeof operator yields the size (in bytes) of its operand, which may be an expression or the parenthesized name of a type. The size is determined from the type of the operand. The result is an integer. If the type of the operand is a variable length array type, the operand is evaluated; otherwise, the operand is not evaluated and the result is an integer constant.

In short: variable length arrays, run at runtime. (Note: Variable Length Arrays are a specific feature -- not arrays allocated with malloc(3).) Otherwise, only the type of the expression is computed, and that at compile time.

Brittabrittain answered 22/11, 2011 at 11:13 Comment(0)
G
36

sizeof is a compile-time built-in operator and is not a function. This becomes very clear in the cases you can use it without the parentheses:

(sizeof x)  // This also works
Gargoyle answered 22/11, 2011 at 13:32 Comment(3)
But how is this an answer to the question?Denature
@phresnel: This is just to make it clear that sizeof is "weird" and is not subject to the rules of normal functions. I edited the post anyway to remove the possible confusion with normal runtime operators like (+) and (-)Gargoyle
The sizeof operator is not a compile-time operator, you only have to give it a VLA to figure this out.Raye
A
25

Note

This answer was merged from a duplicate, which explains the late date.

Original

Except for variable length arrays (VLA) sizeof does not evaluate its arguments. We can see this from the draft C99 standard section 6.5.3.4 The sizeof operator paragraph 2 which says:

The sizeof operator yields the size (in bytes) of its operand, which may be an expression or the parenthesized name of a type. The size is determined from the type of the operand. The result is an integer. If the type of the operand is a variable length array type, the operand is evaluated; otherwise, the operand is not evaluated and the result is an integer constant.

A comment (now removed) asked whether something like this would evaluate at run-time:

sizeof(char[x++]);

and indeed it would, something like this would also work (see them both live):

sizeof(char[func()]);

since they are both variable length arrays. Although, I don't see much practical use in either one.

Note, variable length arrays are covered in the draft C99 standard section 6.7.5.2 Array declarators paragraph 4:

[...] If the size is an integer constant expression and the element type has a known constant size, the array type is not a variable length array type; otherwise, the array type is a variable length array type.

In C11, the answer changes for the VLA case, in certain cases it is unspecified whether the size expression is evaluated or not. From section 6.7.6.2 Array declarators which says:

[...]Where a size expression is part of the operand of a sizeof operator and changing the value of the size expression would not affect the result of the operator, it is unspecified whether or not the size expression is evaluated.

For example, in a case like this (see it live):

sizeof(int (*)[x++])
Aardwolf answered 24/2, 2014 at 18:14 Comment(7)
In what way would sizeof (char[x++]); use the value of x for anything other than determining the value of the expression x++ and the new value for x, both of which are normal with that operator?Kozloski
@CorleyBrigman the snappy answer would be b/c the standard says so, but the reason is b/c we don't know the array size at compile time so we have to evaluate the expression at run-time. VLAs are an interesting topic, here are two posts I have on them here and here.Aardwolf
@Shafik - ahh, ok, i guess my confusion was I didn't realize char[x++] is a VLA. it looks effectively like a char* to my unacquainted eyes.Counterstatement
@CorleyBrigman I don't think a lot of people would realize that considering how questions and answers on VLA usually get higher upvoted.Aardwolf
"and changing the value of the size expression would not affect the result of the operator" - "would not affect"? But in case with sizeof(char[x++]) it would affect. So what did they changed? Cases like sizeof(char[1+0*x++])?Advanced
@Qwertiy: An example of a sizeof expression which involves a VLA, but where where evaluating a sub-expression wouldn't affect the result of the operator would be sizeof (int(*foo[bar()])) since the type being evaluated will always be a pointer to some kind of array, and thus would always be the same size as a pointer to the array's element type.Kozloski
This answer has nothing to do with the question. The questions code causes UB and that should be addressed. The topic about evaluation in sizeof should be addressed in a different question where presented code does not cause UB.Halftruth
C
11

The execution cannot happen during compilation. So ++i/i++ will not happen. Also sizeof(foo()) will not execute the function but return correct type.

Cruzcruzado answered 22/11, 2011 at 14:37 Comment(5)
"The execution cannot happen during compilation." what do you mean?Finedrawn
Compilation will only create object code... The object code will be executed only when the user executes the binary. As sizeof happens at compile time assuming i++ will increment is wrong.Cruzcruzado
"As sizeof happens at compile time" you mean: "as sizeof is a compile time constant expression"?Finedrawn
Like "#define" happens during pre-processing, similarly sizeof will happen at compile time. During compilation all the type information is available so sizeof is evaluated then and there during compilation and value is replaced. As already mentioned by @Toxinantitoxin "From the C99 Standard" before.Cruzcruzado
"sizeof will happen at compile time" for something that is not a variable length arrayFinedrawn
G
11

As the operand of sizeof operator is not evaluated, you can do this:

int f(); //no definition, which means we cannot call it

int main(void) {
        printf("%d", sizeof(f()) );  //no linker error
        return 0;
}

Online demo : http://ideone.com/S8e2Y

That is, you don't need define the function f if it is used in sizeof only. This technique is mostly used in C++ template metaprogramming, as even in C++, the operand of sizeof is not evaluated.

Why does this work? It works because the sizeof operator doesn't operate on value, instead it operates on type of the expression. So when you write sizeof(f()), it operates on the type of the expression f(), and which is nothing but the return type of the function f. The return type is always same, no matter what value the function would return if it actually executes.

In C++, you can even this:

struct A
{
  A(); //no definition, which means we cannot create instance!
  int f(); //no definition, which means we cannot call it
};

int main() {
        std::cout << sizeof(A().f())<< std::endl;
        return 0;
}

Yet it looks like, in sizeof, I'm first creating an instance of A, by writing A(), and then calling the function f on the instance, by writing A().f(), but no such thing happens.

Demo : http://ideone.com/egPMi

Here is another topic which explains some other interesting properties of sizeof:

Geneva answered 23/11, 2011 at 4:35 Comment(3)
Your code causes UB just like the ones from the OP: printf("%d", sizeof(f()) ); This lines causes UB. When you enable compiler warnings it is likely that the compiler warns you about this line.Halftruth
@12431234123412341234123: how and why?Geneva
"%d" expects a int argument. AFAIK a unsigned int with a value <=INT_MAX is also valid. size_t does not have to be the same as an int. Use "%zu" for size_t.Halftruth
G
3

sizeof runs at compile-time, but x++ can only be evaluated at run-time. To solve this, the C++ standard dictates that the operand of sizeof is not evaluated. The C Standard says:

If the type of the operand [of sizeof] is a variable length array type, the operand is evaluated; otherwise, the operand is not evaluated and the result is an integer constant.

Graphology answered 30/11, 2018 at 1:55 Comment(2)
There is no VLA in C++.Chirography
VLA = variable-length arrayDepopulate
H
0

This line here:

printf("%d and ", sizeof(x++)); // note 1

causes undefined behaviour (UB). %d Expects the type int not size_t. After you get UB, the behavior is undefined, including the bytes written to stdout.

If you would fix that by either replacing %d with %zu or casting the value to int, but not both, you would still not increase x, but that is a different problem and should be asked in a different question.

Halftruth answered 2/8, 2021 at 12:8 Comment(0)
A
-1

The sizeof() operator gives size of the data-type only. It does not evaluate inner elements.

Abdel answered 22/1, 2013 at 9:52 Comment(2)
This is false, the sizeof() operator acts recursively, and will get the size in bytes of all elements of a container, members of a class or structure, etc. You can prove this to yourself very easily by creating a simple class with a few members and calling sizeof() on it. (However, anything in there that is a pointer it cannot see the size of - just the size of the pointer.) This happens all at compile time, as other commenters stated: expressions inside the sizeof() do not get evaluated.Kaylee
The operator is sizeof, not sizeof().Formicary

© 2022 - 2024 — McMap. All rights reserved.