[PATCH] powerpc: Fix POWER7/PPC64 performance regression on LE

Richard Henderson rth@twiddle.net
Tue Jan 13 20:12:00 GMT 2015


On 01/13/2015 11:43 AM, Adhemerval Zanella wrote:
> +	std	rWORD8, rWORD8SAVE(r1)
> +	cfi_offset(rWORD8, rWORD8SAVE)
> +	std	rWORD7, rWORD7SAVE(r1)
> +	cfi_offset(rWORD7, rWORD7SAVE)
> +	std	rOFF8, rOFF8SAVE(r1)
> +	cfi_offset(rWORD7, rOFF8SAVE)
> +	std	rOFF16, rOFF16SAVE(r1)
> +	cfi_offset(rWORD7, rOFF16SAVE)
> +	std	rOFF24, rOFF24SAVE(r1)
> +	cfi_offset(rWORD7, rOFF24SAVE)
> +	std	rOFF32, rOFF32SAVE(r1)
> +	cfi_offset(rWORD7, rOFF32SAVE)

Given a sequence of stores like this, it's better to
put all of the cfi_offset annotations after the last store.

By interspersing them like this, you're just adding unnecessary
DW_CFA_advance opcodes to the unwind info.


r~



More information about the Libc-alpha mailing list