On Thu, 12 Jan 2012 17:20:06 +0100, Peter Meerwald <[email protected]> wrote: > +void pa_remap_func_init_neon(pa_cpu_arm_flag_t flags) { > +#if defined (__ARM_NEON__) > + > +#ifdef RUN_TEST > + run_test_float(); > + run_test_s16(); > +#endif > + > + if (flags & PA_CPU_ARM_NEON) { > + pa_log_info("Initialising ARM NEON optimized remappers."); > + pa_set_init_remap_func((pa_init_remap_func_t) init_remap_neon); > + }
This blob does not seem to make much sense. If __ARM_NEON__ is defined, then the compiler assumes NEON is supported. So a run-time check is silly. > + > +#endif /* defined (__ARM_NEON__) */ > +} -- Rémi Denis-Courmont http://www.remlab.net/ _______________________________________________ pulseaudio-discuss mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
