UefiCpuPkg/BaseUefiCpuLib: Use NASM read-only data section name
authorMichael Kinney <michael.d.kinney@intel.com>
Tue, 16 May 2017 20:35:17 +0000 (13:35 -0700)
committerMichael Kinney <michael.d.kinney@intel.com>
Fri, 19 May 2017 21:07:33 +0000 (14:07 -0700)
https://bugzilla.tianocore.org/show_bug.cgi?id=556

NASM requires read-only data sections to use the section
name .rodata.  This fix changes .rdata to .rodata.

The build failure from use of .rdata is seen when using
the XCODE5 tool chain.

Section "7.8.1 macho extensions to the SECTION Directive"
of the NASM documentation at http://www.nasm.us/doc/
describes the section name requirements.

Cc: Jeff Fan <jeff.fan@intel.com>
Cc: Andrew Fish <afish@apple.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Michael Kinney <michael.d.kinney@intel.com>
Reviewed-by: Jeff Fan <jeff.fan@intel.com>
UefiCpuPkg/Library/BaseUefiCpuLib/Ia32/InitializeFpu.nasm
UefiCpuPkg/Library/BaseUefiCpuLib/X64/InitializeFpu.nasm

index 55085e019f46b1493aab1377111c227657c6a020..6dac4ccfc7a8888165ce4c5d43c2f16191d8ab83 100644 (file)
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------\r
 ;*\r
-;*   Copyright (c) 2016, Intel Corporation. All rights reserved.<BR>\r
+;*   Copyright (c) 2016 - 2017, Intel Corporation. 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
@@ -12,7 +12,7 @@
 ;*\r
 ;------------------------------------------------------------------------------\r
 \r
-    SECTION .rdata\r
+    SECTION .rodata\r
 \r
 ;\r
 ; Float control word initial value:\r
index 904b64e5106025f5575d822b4f1a7b9606654d3e..404c199b35925dc2513b545ced88c76be0446a19 100644 (file)
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------\r
 ;*\r
-;*   Copyright (c) 2009 - 2012, Intel Corporation. All rights reserved.<BR>\r
+;*   Copyright (c) 2009 - 2017, Intel Corporation. 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
@@ -12,7 +12,7 @@
 ;*\r
 ;------------------------------------------------------------------------------\r
 \r
-    SECTION .rdata\r
+    SECTION .rodata\r
 ;\r
 ; Float control word initial value:\r
 ; all exceptions masked, double-extended-precision, round-to-nearest\r