MdeModulePkg: Remove NORETURN for PeiCore() and DxeMain() function

PeiCore EntryPoint library _ModuleEntryPoint() will call PeiCore(), then call
CpuDeadLoop (). When NORETURN is added for PeiCore(), MSVC compiler will report
warning C4702: unreachable code for CpuDeadLoop (). And, the warning is treated
as error and cause build break. DxeMain() has the similar issue.

edk2 uses EntryPoint library to wrap every module entry point function except
for SEC. The module entry point is still called by _ModuleEntryPoint(). So,
there will be negative impact to add NORETURN for the module entry point.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Liming Gao <liming.gao@intel.com>

MdeModulePkg/Core/Dxe/DxeMain/DxeMain.c

@@ -235,7 +235,6 @@ GLOBAL_REMOVE_IF_UNREFERENCED EFI_LOAD_FIXED_ADDRESS_CONFIGURATION_TABLE    gLoa
 
 **/
 VOID
-NORETURN
 EFIAPI
 DxeMain (
   IN  VOID *HobStart

MdeModulePkg/Core/Pei/PeiMain.h

@@ -315,7 +315,6 @@ typedef struct {
 
 **/
 VOID
-NORETURN
 EFIAPI
 PeiCore (
   IN CONST EFI_SEC_PEI_HAND_OFF        *SecCoreData,

MdeModulePkg/Core/Pei/PeiMain/PeiMain.c

@@ -136,7 +136,6 @@ ShadowPeiCore (
 
 **/
 VOID
-NORETURN
 EFIAPI
 PeiCore (
   IN CONST EFI_SEC_PEI_HAND_OFF        *SecCoreDataPtr,