Subject: Re: CVS commit: src/sys/arch/arm/arm32
To: Steve Woodford <scw@netbsd.org>
From: Richard Earnshaw <rearnsha@arm.com>
List: source-changes
Date: 10/16/2003 09:02:51
> On Wednesday 15 October 2003 2:56 pm, Richard Earnshaw wrote:
>
> > This is wrong. ldrd and strd are available on any v5E processor, so
> > it shouldn't be wrapped in '#ifdef XSCALE'.
>
> Ok, my 'fix' commit also jumped the gun, and hasn't actually solved the
> problem.
>
> What's the easiest/quickest way to determine if strd/ldrd are available?
>
> Given that the code is in a critical path, a compile-time check would be
> best, otherwise is there a quick way to check the cpu id?
>
> Cheers, Steve
>
Isn't ok just to check for it unconditionally?
This whole section really needs rewriting for modern ARM processors
anyway, it doesn't handle Thumb and it wouldn't stand a chance with
Jazelle.
R.