It might be a good idea to align the new_heap pointer, as it happens for example here:

https://github.com/mbedmicro/mbed/blob/master/libraries/mbed/targets/cmsis/TARGET_NXP/TARGET_LPC176X/TOOLCHAIN_GCC_CS/sys.cpp

Not sure if this is a hard requirement, but it seems to be a bit safer.