ArmPkg/DefaultExceptionHandlerLib: put ASSERT (FALSE) last

Putting DEBUG () code after an ASSERT (FALSE) statement is not very
useful, since the code will be unreachable on DEBUG builds and compiled
out on RELEASE builds. So move the ASSERT () statement after it.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Acked-by: Mark Rutland <mark.rutland@arm.com>
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
This commit is contained in:
Ard Biesheuvel 2016-07-01 12:29:37 +02:00
parent 87f66b63d4
commit f9ebccec09
1 changed files with 1 additions and 1 deletions

View File

@ -191,7 +191,6 @@ DefaultExceptionHandler (
DEBUG ((EFI_D_ERROR, "\n ESR : EC 0x%02x IL 0x%x ISS 0x%08x\n", (SystemContext.SystemContextAArch64->ESR & 0xFC000000) >> 26, (SystemContext.SystemContextAArch64->ESR >> 25) & 0x1, SystemContext.SystemContextAArch64->ESR & 0x1FFFFFF ));
DescribeExceptionSyndrome (SystemContext.SystemContextAArch64->ESR);
ASSERT (FALSE);
DEBUG ((EFI_D_ERROR, "\nStack dump:\n"));
for (Offset = -256; Offset < 256; Offset += 32) {
@ -204,5 +203,6 @@ DefaultExceptionHandler (
*(UINT64 *)(SystemContext.SystemContextAArch64->SP + Offset + 24)));
}
ASSERT (FALSE);
CpuDeadLoop ();
}