Thanks Brian. Is this also in clang for ARM?
On Wed, Jun 14, 2017 at 4:16 PM, Brian Brooks <[email protected]> wrote: > Reviewed-by: Brian Brooks <[email protected]> > > ACLE [1] was added to GCC 4.9. GCC 4.8 does define __aarch64__ > so this change should be safe. > > [1] > http://infocenter.arm.com/help/topic/com.arm.doc.ihi0053c/IHI0053C_acle_2_0.pdf > > On 06/21 14:48:36, Petri Savolainen wrote: >> Use __aarch64__ instead of __ARM_ARCH, since it's backwards >> compatible between GCC versions. >> >> Fixes bug https://bugs.linaro.org/show_bug.cgi?id=3066 >> >> Signed-off-by: Petri Savolainen <[email protected]> >> --- >> platform/linux-generic/arch/arm/odp_cpu_arch.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/platform/linux-generic/arch/arm/odp_cpu_arch.c >> b/platform/linux-generic/arch/arm/odp_cpu_arch.c >> index 91d439d9..fd1b3ed6 100644 >> --- a/platform/linux-generic/arch/arm/odp_cpu_arch.c >> +++ b/platform/linux-generic/arch/arm/odp_cpu_arch.c >> @@ -68,7 +68,7 @@ int cpu_has_global_time(void) >> >> uint64_t cpu_global_time(void) >> { >> -#if __ARM_ARCH == 8 >> +#if __aarch64__ >> uint64_t cntvct; >> >> /* >> @@ -88,7 +88,7 @@ uint64_t cpu_global_time(void) >> >> uint64_t cpu_global_time_freq(void) >> { >> -#if __ARM_ARCH == 8 >> +#if __aarch64__ >> uint64_t cntfrq; >> >> __asm__ volatile("mrs %0, cntfrq_el0" : "=r"(cntfrq) : : ); >> -- >> 2.13.0 >>
