On Thu, Jul 16, 2015 at 04:48:43PM +0100, Kyrill Tkachov wrote: > Sorry, had sent out the wrong version. > This is the right patch. > > Thanks, > Kyrill > > On 16/07/15 16:20, Kyrill Tkachov wrote: > > Hi all, > > > > This patch transforms the Cortex-A53 erratum 835769 workaround checks into > > a macro. > > This way we don't have to override aarch64_fix_a53_err835769 in the default > > case > > and this allows us to keep track of when the user doesn't specify this > > option, > > which may come in handy later on when we decide the inlining rules. > > > > This patch also makes TARGET_FIX_ERR_A53_835769_DEFAULT unconditionally > > defined to > > 0 or 1, so that we don't have to check it if #ifdefs. > > > > Bootstrapped and tested as part of series on aarch64. > > Checked that the workaround is applied as previously. > > > > Ok for trunk?
OK. Thanks, James > > > > 2015-07-16 Kyrylo Tkachov <kyrylo.tkac...@arm.com> > > > > * config/aarch64/aarch64.h (TARGET_FIX_ERR_A53_835769_DEFAULT): Always > > define to 0 or 1. > > (TARGET_FIX_ERR_A53_835769): New macro. > > * config/aarch64/aarch64.c (aarch64_override_options_internal): Remove > > handling of opts->x_aarch64_fix_a53_err835769. > > (aarch64_madd_needs_nop): Check for TARGET_FIX_ERR_A53_835769 rather > > than aarch64_fix_a53_err835769. > > * config/aarch64/aarch64-elf-raw.h: Update for above changes. > > * config/aarch64/aarch64-linux.h: Likewise. >
