On 08/05/2016 12:18 AM, Andrew Pinski wrote:
This patch disables the forming of the load/store pairs for SImode if
we are tuning for ThunderX. I used the tuning flags route so it can
be overridden if needed later on or if someone else wants to use the
same method for their core.
+ if (mode == SImode
+ && AARCH64_EXTRA_TUNE_SLOW_LDPW
+ && !optimize_size)
+ return false;
AARCH64_EXTRA_TUNE_SLOW_LDPW is a non-zero bit-mask. That will always
be true. This is present in two places in the patch. You need
something more like
&& (aarch64_tune_params.extra_tuning_flags
& AARCH64_EXTRA_TUNE_SLOW_LDPW)
You should verify that the patch disables the optimization for ThunderX
but does not disable it for other targets.
Jim
