svolume: Mark channel parameter as earlyclobber
authorArun Raghavan <arun@arunraghavan.net>
Thu, 11 Jul 2019 05:27:31 +0000 (10:57 +0530)
committerTanu Kaskinen <tanuk@iki.fi>
Mon, 15 Jul 2019 13:50:16 +0000 (13:50 +0000)
commite8fe04b2f6ab248a09176622b062bd934eb49e4c
tree82e8c6f498b433cb88e20c74addf607419489628
parente34dd0fb8f8ca8e66fc426e2524565c9ed4c8794
svolume: Mark channel parameter as earlyclobber

For all our MMX/SSE code, we use a temporary channel variable, assigned
to the DI register, which is zero'ed as the very first operation in the
inline assembly code, before any other code is run.

With GCC 9.1, while using -O2, the DI register is also used for the
input operand. This is perfectly legal, but causes our code to become
incorrect because the output operand that is assigned to DI is not
explicitly marked as being clobbered before inputs are read.

This change fixes the problem by adding an earlyclobber annotation (&)
to the DI output argument.
src/pulsecore/svolume_mmx.c
src/pulsecore/svolume_sse.c