diff options
author | Ard Biesheuvel <ard.biesheuvel@linaro.org> | 2016-03-17 11:33:58 +0100 |
---|---|---|
committer | Ard Biesheuvel <ard.biesheuvel@linaro.org> | 2016-03-22 14:39:02 +0100 |
commit | 8a771a2e39771b950823abcd3786e82cfd8e6e27 (patch) | |
tree | 5588b7dbbb6c7d546d705983c66b40d12de00834 | |
parent | 2187f9a0523c44604a27bbf6563d5c373e73cc31 (diff) | |
download | edk2-platforms-8a771a2e39771b950823abcd3786e82cfd8e6e27.tar.xz |
ArmPkg/ArmExceptionLib: make build time define visible to the compiler
The global gArmRelocateVectorTable is a build time constant, but due to
its external linkage and lack of constness, the compiler does not see that.
So turn it into a static boolean, and at the same time, make the function
CopyExceptionHandlers() (which is only called if gArmRelocateVectorTable is
set) static as well, so that the compiler can eliminate it completely if
we are using the vector table in place.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
Reviewed-by: Eugene Cohen <eugene@hp.com>
-rw-r--r-- | ArmPkg/Library/ArmExceptionLib/ArmExceptionLib.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/ArmPkg/Library/ArmExceptionLib/ArmExceptionLib.c b/ArmPkg/Library/ArmExceptionLib/ArmExceptionLib.c index 5977a3e8fa..0cf0766b9c 100644 --- a/ArmPkg/Library/ArmExceptionLib/ArmExceptionLib.c +++ b/ArmPkg/Library/ArmExceptionLib/ArmExceptionLib.c @@ -26,6 +26,7 @@ #include <Library/DebugLib.h>
#include <Library/DefaultExceptionHandlerLib.h>
+STATIC
RETURN_STATUS
CopyExceptionHandlers(
IN PHYSICAL_ADDRESS BaseAddress
@@ -66,9 +67,9 @@ extern UINTN gDebuggerNoHandlerValue; // library we cannot represent this in a PCD since PCDs are evaluated on
// a per-module basis.
#if defined(ARM_RELOCATE_VECTORS)
-BOOLEAN gArmRelocateVectorTable = TRUE;
+STATIC CONST BOOLEAN gArmRelocateVectorTable = TRUE;
#else
-BOOLEAN gArmRelocateVectorTable = FALSE;
+STATIC CONST BOOLEAN gArmRelocateVectorTable = FALSE;
#endif
@@ -151,6 +152,7 @@ with default exception handlers. @retval EFI_UNSUPPORTED This function is not supported.
**/
+STATIC
RETURN_STATUS
CopyExceptionHandlers(
IN PHYSICAL_ADDRESS BaseAddress
|