Developer-Guide/standards/cpp.md
Dennis Eichhorn 983a6d4599
Update cpp.md
Signed-off-by: Dennis Eichhorn <spl1nes.com@googlemail.com>
2024-08-05 19:38:08 +02:00

3.7 KiB
Executable File

C/C++

The C/C++ code should focus on using simplicity over "modern solutions". This may often mean to heavily restrict the code. The following rule of thumb applies:

  1. C99 and C++11 should be used where reasonable
  2. C++ may be used in places where external libraries basically require C++

The reason for the strong focus on C is that we personally believe that C is simpler and easier to understand than the various abstractions provided by C++.

Operating system support

C/C++ solutions should be valid on Windows 10+ and Linux.

Performance

When writing code keep the following topics in mind:

  • Branching / Branchless programming
  • Instruction tables and their latency / throughput
  • Cache Sizes
  • Cache Line Size
  • Cache Locality
  • Cache Associativity
  • Memory Bandwidth
  • Memory Latency
  • Prefetching
  • Alignment / Packing
  • Array of Structs vs Struct of Arrays
  • SIMD
  • Choosing correct data types
    • Data Type Sizes (e.g. 32 bit vs 64 bit)
    • Containers (e.g. arrays vs vectors)
    • Signed vs unsigned
  • Threading
  • Cost of abstractions
  • atomics vs locking (mutex)
  • Cache line sharing between CPU cores

Branching / Branchless programming

Branched code

for (int i = 0; i < N; i++)
    if (a[i] < 50) {
        s += a[i];
    }
}

Branchless code

for (int i = 0; i < N; i++)
    s += (a[i] < 50) * a[i];
}

Instruction table latency

Instruction Latency RThroughput
jmp - 2
mov r, r - 1/4
mov r, m 4 1/2
mov m, r 3 1
add 1 1/3
cmp 1 1/4
popcnt 1 1/4
mul 3 1
div 13-28 13-28

https://www.agner.org/optimize/instruction_tables.pdf

Cache line sharing between CPU cores

When working with multi-threading you may choose to use atomic variables and atomic operations to reduce the locking in your application. You may think that a variable value a[0] used by thread 1 on core 1 and a variable value a[1] used by thread 2 on core 2 will have no performance impact. However, this is wrong. Core 1 and core 2 both have different L1 and L2 caches BUT the CPU doesn't just load individual variables, it loads entire cache lines (e.g. 64 bytes). This means that if you define int a[2], it has a high chance of being on the same cache line and therfore thread 1 and thread 2 both have to wait on each other when doing atomic writes.

Namespace

use

Namespaces must never be globally used. This means for example use namespace std; is prohibited and functions from the standard namespace should be prefixed instead std::

Templates

Don't use C++ templates.

Allocation

Use memory arenas instead of over and over manually allocating memory.

However, if neccessary use C allocation methods for heap allocation.

Functions

C++ function

Don't use C++ standard library functions or C++ functions provided by other C++ header files unless you have to work with C++ types which is often required when working with third party libraries.

Parameters

Generally, functions that take pointers to non-scalar types should modify the data instead of allocating new memory IF reasonable. This forces programmers to consciously create copies before passing data IF they need the original data. To indicate that a reference/pointer is not modified by a function define them as const!

We believe this approach provides a framework for better memory management and better performance in general.

Examples for this can be:

  • Matrix multiplication with a scalar
  • Sorting data (depends on sorting algorithm)