On 14/08/15 10:23, James Greenhalgh wrote: > > Hi, > > I spotted that these are missing from the "is_neon_type" attribute: > > neon_fp_abs_s, neon_fp_abs_s_q, neon_fp_abs_d, neon_fp_abs_d_q, > neon_fp_neg_s, neon_fp_neg_s_q, neon_fp_neg_d, neon_fp_neg_d_q, > neon_fp_to_int_d, neon_fp_to_int_d_q, > neon_int_to_fp_d, neon_int_to_fp_d_q, > > This patch adds them. In principle this could have shown up by allowing > us to create a limited set of conditional neon instructions, or as a > scheduling quirk in the Cortex-A7 and Cortex-A53 scheduling models, but I > didn't see any impact in some smoke testing. > > Bootstrapped on arm-none-linux-gnueabihf and aarch64-none-linux-gnu with > no issues.
This is OK - Thanks for catching this. Ramana > > OK? > > Thanks, > James > > --- > 2015-08-14 James Greenhalgh <james.greenha...@arm.com> > > * config/arm/types.md (is_neon_type): Add missing types. >