Re: [PATCH][AArch32] Fix the build for M class semihosting

Corinna Vinschen <[email protected]>
Newsgroups gmane.comp.lib.newlib
Message-ID <[email protected]>
On Feb  8 18:34, Tamar Christina wrote:
> Hi All,
> 
> 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
> always.
> 
> Regtested on arm-none-eabi and no issues, using a build with m class
> multilibs too.
> 
> Ok for master?

Pushed.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEEoVYPmneWZnwT6kwF9TYGna5ET6AFAlxhYYAACgkQ9TYGna5E
T6CfTQ/+IYW9Ys5KjtRd4Q8TjzVIMRnKnFXadJ1ugyUN+dcgZVMWhcWr4qatwTPQ
/yxDWseytcLDElqFMcNtrQFtkTumZmW7LEuSk/EkkzrBZJ/yXBJZaRUxMJpKB4PF
erMRsTozcEiOVpVIwTm0cBnfoJzHQuXjRgoyTt/Uv7FAv2tUqZBbmYq3y8tHnRda
CjKHN/p2aroVzf91AHFz0yadBtR0OXnLy2CAh+G4YGo5jrrLfp8+9oBybg/qJPsY
ZZQyEfrj6JKmCgMdtl/0wD4Qp21mej3B3tIOGLL2FjQHTJs2IhYuFsq7GF7V/Htv
gsMeySgS2nPuNEdstVdxmDeJ8SBbrlN5Atw4YfLCv0FsoQrya28MtWlyII3sBWLD
oetcgRkqTOqBtbi6vsqS+ou2Sso4beQQ41pPItkdObI4TC98T+st/NvkyPu9hClT
1wxlLfoR0ZUzRqjmk6N4hpFj/iRPcDhAXLHbwg7j3uZlzUiw6qBv2rdpHhStTuBq
ICeupnrLOBCXZzLMkORbnjpYoC3RZIXzsIoA50H0du0IoeNwZohuRAzRvsLC6kaJ
iZDIF2DVRWoE7g3E1oymxc/l0Crd1cBI8YR8SoABmAxshrCoLpQV90zraio6/r1t
qJgt176ZykwAmT9JibID96SQBrP5bIlLZBHPBJu9c3xxsL9xqtU=
=+Sgd
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.