Function parameter or local variable

Question

Let's say I have a loop which repeats millions of times. Inside of this loop I have a function call. Inside of this function I need to operate on some temporary variable created at the very beginning. Now, which one is better: a) Create temporary variable at the beginning of the code, initialize it at the beginning of the loop, and pass it as function parameter b) Create just local temporary variable at the beginning of the called function? Is this answerable question? I'd like to know which point is considered better practice, or which one is faster.

Accepted Answer

Let's throw up some possible definitions for `some_function()`, the function you will be calling from your loop.

```
// Method 1
void some_function() {
    int temporary;

// Use temporary
}

// Method 2
void some_function(int temporary) {
    // Use temporary
}

// Method 3
void some_function(int *temporary) {
    // Use *temporary
}
```

Method 1 is likely to be the most readable out of these options, and so it's the one I would prefer unless you have a really good reason to do something else. It is also likely to be faster than either of the others, unless your compiler is inlining the function call. If it is, then all three are likely to perform exactly the same (method 3 might still be slower if the compiler isn't able to optimize away the pointer dereferences).

If the compiler is not inlining, then method 2 is likely to be slower than method 1. This is because, in terms of stack-allocated memory, they are the same -- function arguments are going to be stored on the stack the same way locals are. The only difference between a function argument and a local in this context is that the function argument can be given a value by the caller. This step of transferring the value from the caller to the function is (theoretically) going to slow down the invocation.

Method 3 is almost certainly going to be slower, since accesses to the temporary memory will include a level of indirection. Dereferencing a pointer is not a cheap operation compared to accessing a local.

Of course, if performance is absolutely critical then you should benchmark these approaches. I suspect that method 1 will turn out to be the fastest (or at least no slower than the others) and additionally seems more readable to me.

Function parameter or local variable

Solution

Problem