[Openmp-dev] AArch64 support

[C Bergström]

On Mon, Oct 27, 2014 at 11:20 PM, Peyton, Jonathan L <
jonathan.l.peyton at intel.com> wrote:

>  -if(NOT arch)
>
> -   if("${CMAKE_SIZEOF_VOID_P}" STREQUAL "4")
>
> -       set(arch           32          CACHE STRING "The architecture to
> build for (32e/32/arm/ppc64).  32e is Intel(R) 64 architecture, 32 is IA-32
> architecture")
>
> -   else()
>
> -       set(arch           32e         CACHE STRING "The architecture to
> build for (32e/32/arm/ppc64).  32e is Intel(R) 64 architecture, 32 is IA-32
> architecture")
>
> -   endif()
>
> +#if(NOT arch)
>
> +#  if("${CMAKE_SIZEOF_VOID_P}" STREQUAL "4")
>
> +#      set(arch           32          CACHE STRING "The architecture to
> build for (32e/32/arm/ppc64/aarch64).  32e is Intel(R) 64 architecture, 32
> is IA-32 architecture")
>
> +#  else()
>
> +#      set(arch           32e         CACHE STRING "The architecture to
> build for (32e/32/arm/ppc64/aarch64).  32e is Intel(R) 64 architecture, 32
> is IA-32 architecture")
>
> +#  endif()
>
> +#endif()
>
>
>
> We can get rid of the if(NOT arch) line, this guard is unnecessary.
>

ack - I saw a lot of things I'd like to clean-up, but I didn't touch them
all. I'll include this clean-up in my patch.

Thanks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/openmp-dev/attachments/20141027/5ef275df/attachment.html>