MdePkg: Disable some Level 4 warnings for VS2017/ARM
authorLiming Gao <liming.gao@intel.com>
Wed, 7 Feb 2018 01:25:15 +0000 (09:25 +0800)
committerLiming Gao <liming.gao@intel.com>
Wed, 7 Feb 2018 01:49:20 +0000 (09:49 +0800)
We disable the exact same warnings as IA32 and X64.

Also create a dummy macro for PRESERVE8, as this is not supported by
the Microsoft ARM assembler.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Pete Batard <pete@akeo.ie>
Reviewed-by: Liming Gao <liming.gao@intel.com>
MdePkg/Include/Arm/ProcessorBind.h

index c30d353..8cca0f3 100644 (file)
@@ -1,15 +1,15 @@
 /** @file\r
   Processor or Compiler specific defines and types for ARM.\r
 \r
-  Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.<BR>\r
+  Copyright (c) 2006 - 2018, Intel Corporation. All rights reserved.<BR>\r
   Portions copyright (c) 2008 - 2009, Apple Inc. All rights reserved.<BR>\r
-  This program and the accompanying materials                          \r
-  are licensed and made available under the terms and conditions of the BSD License         \r
-  which accompanies this distribution.  The full text of the license may be found at        \r
-  http://opensource.org/licenses/bsd-license.php                                            \r
+  This program and the accompanying materials\r
+  are licensed and made available under the terms and conditions of the BSD License\r
+  which accompanies this distribution.  The full text of the license may be found at\r
+  http://opensource.org/licenses/bsd-license.php\r
 \r
-  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,                     \r
-  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.             \r
+  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
+  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
 \r
 **/\r
 \r
 #pragma pack()\r
 #endif\r
 \r
+#if defined(_MSC_EXTENSIONS)\r
+\r
+//\r
+// Disable some level 4 compilation warnings (same as IA32 and X64)\r
+//\r
+\r
+//\r
+// Disabling bitfield type checking warnings.\r
+//\r
+#pragma warning ( disable : 4214 )\r
+\r
+//\r
+// Disabling the unreferenced formal parameter warnings.\r
+//\r
+#pragma warning ( disable : 4100 )\r
+\r
+//\r
+// Disable slightly different base types warning as CHAR8 * can not be set\r
+// to a constant string.\r
+//\r
+#pragma warning ( disable : 4057 )\r
+\r
+//\r
+// ASSERT(FALSE) or while (TRUE) are legal constructs so suppress this warning\r
+//\r
+#pragma warning ( disable : 4127 )\r
+\r
+//\r
+// This warning is caused by functions defined but not used. For precompiled header only.\r
+//\r
+#pragma warning ( disable : 4505 )\r
+\r
+//\r
+// This warning is caused by empty (after preprocessing) source file. For precompiled header only.\r
+//\r
+#pragma warning ( disable : 4206 )\r
+\r
+//\r
+// Disable 'potentially uninitialized local variable X used' warnings\r
+//\r
+#pragma warning ( disable : 4701 )\r
+\r
+//\r
+// Disable 'potentially uninitialized local pointer variable X used' warnings\r
+//\r
+#pragma warning ( disable : 4703 )\r
+\r
+#endif\r
+\r
 //\r
-// RVCT does not support the __builtin_unreachable() macro\r
+// RVCT and MSFT don't support the __builtin_unreachable() macro\r
 //\r
-#ifdef __ARMCC_VERSION\r
+#if defined(__ARMCC_VERSION) || defined(_MSC_EXTENSIONS)\r
 #define UNREACHABLE()\r
 #endif\r
 \r
-#if _MSC_EXTENSIONS \r
+#if defined(_MSC_EXTENSIONS)\r
   //\r
   // use Microsoft* C compiler dependent integer width types\r
   //\r
   typedef signed char         INT8;\r
 #else\r
   //\r
-  // Assume standard ARM alignment. \r
+  // Assume standard ARM alignment.\r
   // Need to check portability of long long\r
   //\r
   typedef unsigned long long  UINT64;\r
@@ -126,7 +175,7 @@ typedef INT32   INTN;
 // use the correct C calling convention. All protocol member functions and\r
 // EFI intrinsics are required to modify their member functions with EFIAPI.\r
 //\r
-#define EFIAPI    \r
+#define EFIAPI\r
 \r
 // When compiling with Clang, we still use GNU as for the assembler, so we still\r
 // need to define the GCC_ASM* macros.\r
@@ -147,34 +196,39 @@ typedef INT32   INTN;
 \r
     #define GCC_ASM_EXPORT(func__)  \\r
              .global  _CONCATENATE (__USER_LABEL_PREFIX__, func__)    ;\\r
-             .type ASM_PFX(func__), %function  \r
+             .type ASM_PFX(func__), %function\r
 \r
     #define GCC_ASM_IMPORT(func__)  \\r
              .extern  _CONCATENATE (__USER_LABEL_PREFIX__, func__)\r
-             \r
+\r
   #else\r
     //\r
-    // .type not supported by Apple Xcode tools \r
+    // .type not supported by Apple Xcode tools\r
     //\r
-    #define INTERWORK_FUNC(func__)  \r
+    #define INTERWORK_FUNC(func__)\r
 \r
     #define GCC_ASM_EXPORT(func__)  \\r
              .globl  _CONCATENATE (__USER_LABEL_PREFIX__, func__)    \\r
-  \r
-    #define GCC_ASM_IMPORT(name)  \r
+\r
+    #define GCC_ASM_IMPORT(name)\r
 \r
   #endif\r
+#elif defined(_MSC_EXTENSIONS)\r
+  //\r
+  // PRESERVE8 is not supported by the MSFT assembler.\r
+  //\r
+  #define PRESERVE8\r
 #endif\r
 \r
 /**\r
   Return the pointer to the first instruction of a function given a function pointer.\r
-  On ARM CPU architectures, these two pointer values are the same, \r
+  On ARM CPU architectures, these two pointer values are the same,\r
   so the implementation of this macro is very simple.\r
-  \r
+\r
   @param  FunctionPointer   A pointer to a function.\r
 \r
   @return The pointer to the first instruction of a function given a function pointer.\r
-  \r
+\r
 **/\r
 #define FUNCTION_ENTRY_POINT(FunctionPointer) (VOID *)(UINTN)(FunctionPointer)\r
 \r