#1628: Enable padding by default ------------------------------------+--------------------------------------- Reporter: eschnett | Owner: Type: enhancement | Status: confirmed Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: development version Resolution: | Keywords: ------------------------------------+---------------------------------------
Comment (by eschnett):
It seems that padding is currently broken, in the sense that simulation results differ. To make this happen, someone needs to build with padding enabled, and then track down where the differences come from.
Aligning the first non-ghost point is straightforward.
Unaligned memory access is fast on Intel system, but it requires additional instructions. Since McLachlan is limited by instruction cache size, ensuring aligned access may have a larger benefit than one would otherwise expect. Also, with AVX instructions, unaligned access has become more expensive.