JehanneOS/newlib
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

AArch32: Fix the build for M class semihosting

Browse Source 
The M class cores don't support Semihosting v2 mixed mode, but we were
accidentally using the new immediates for it.  My last patch changed the
immediates which broke the build because doing a full multi-lib build
including M architectures now results in an assembler error instead of
silently doing the wrong thing.

This fixes the issue by changing the defines around such that According
to the specs any M class build uses the normal semihosting instructions.

Regtested on arm-none-eabi and no issues, using a build with m class
multilibs too.
This commit is contained in:
Tamar Christina 2019-02-08 17:13:24 +00:00 committed by Corinna Vinschen
parent 52d91f112e
commit 1ba66fe8fa
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
libgloss/arm/swi.h
View File

@ -30,7 +30,9 @@
/* Now the SWI numbers and reason codes for RDI (Angel) monitors. */
#if defined (SEMIHOST_V2) && defined (SEMIHOST_V2_MIXED_MODE)
#if defined (SEMIHOST_V2) \
&& defined (SEMIHOST_V2_MIXED_MODE) \
&& !defined (THUMB_VXM)
#define AngelSWI_ARM 0xE10F0070 /* HLT #0xF000 A32. */
#ifdef __thumb__
#define AngelSWI 0xBABC /* HLT #0x3c T32. */