SPEC2006: Pronounce endianness flags both ways

[nwf]

Summary

473.astar chose to pronounce the CPU endianness C preprocessor symbol differently from the rest of the benchmark suite.

This was caught in the make-based build system, but missed in the cmake-based system.

[naromero77]

Thanks for catching this.

Just to double-check, so you need BOTH spellings?

[Meinersbur]

LGTM

In D96432#2554558, @naromero77 wrote:

Just to double-check, so you need BOTH spellings?

SPEC_CPU_BIG_ENDIAN for 473.astar, SPEC_CPU_BIGENDIAN for all others.

[Meinersbur]

ping?

Do you need help with pushing this to main?

[jrtc27]

In D96432#2862070, @Meinersbur wrote:

ping?

Do you need help with pushing this to main?

Wes isn't a committer; I could do it for him but don't have a clone to hand so yes, please do commit on his behalf.

[Meinersbur]

OK. wil do.

External/SPEC/SpecCPU2006.cmake

if(ARCH STREQUAL "x86")
     else()
       list(APPEND CPPFLAGS -DSPEC_CPU_IA32)
     endif()
   endif()
 
   include(TestBigEndian)
   test_big_endian(IS_BIGENDIAN)
   if(IS_BIGENDIAN)
-    list(APPEND CPPFLAGS -DSPEC_CPU_BIGENDIAN)
+    list(APPEND CPPFLAGS -DSPEC_CPU_BIGENDIAN -DSPEC_CPU_BIG_ENDIAN)
   else()
-    list(APPEND CPPFLAGS -DSPEC_CPU_LITTLEENDIAN)
+    list(APPEND CPPFLAGS -DSPEC_CPU_LITTLEENDIAN -DSPEC_CPU_LITTLE_ENDIAN)
   endif()
 
   if(CMAKE_SIZEOF_VOID_P EQUAL 8)
     list(APPEND CPPFLAGS -DSPEC_CPU_LP64)
   endif()
 
   if(TARGET_OS STREQUAL "Darwin")
     # Work around built in -Werror=implicit-function-declaration default on iOS