This is the mail archive of the newlib@sourceware.org mailing list for the newlib project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

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

From: Corinna Vinschen <vinschen at redhat dot com>
To: Tamar Christina <Tamar dot Christina at arm dot com>
Cc: "newlib at sourceware dot org" <newlib at sourceware dot org>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>, Alexander Fedotov <alfedotov at gmail dot com>, nd <nd at arm dot com>
Date: Mon, 11 Feb 2019 12:50:25 +0100
Subject: Re: [PATCH][Newlib][AArch32] Fix the build for M class semihosting
References: <DB6PR0802MB23094BE16F74E11BF188F735FF690@DB6PR0802MB2309.eurprd08.prod.outlook.com>
Reply-to: newlib at sourceware dot org

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

Attachment: signature.asc
Description: PGP signature

References:
- [PATCH][Newlib][AArch32] Fix the build for M class semihosting
  - From: Tamar Christina

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]