[FFmpeg-devel] [PATCH] read_time() for SPARC

[Michael Niedermayer]

On Thu, Sep 09, 2010 at 10:01:44PM +0100, M?ns Rullg?rd wrote:
> Michael Niedermayer <michaelni at gmx.at> writes:
> 
> > On Wed, Sep 08, 2010 at 01:08:57AM +0400, Michael Kostylev wrote:
> >> 
> >> The attached patch provides access to a 63-bit tick counter which is
> >> available on the v9 cpus in the manner compatible with the v8+ abi.
> >> 
> >> Michael
> >
> >>  sparc/timer.h |   41 +++++++++++++++++++++++++++++++++++++++++
> >>  timer.h       |    2 ++
> >>  2 files changed, 43 insertions(+)
> >> 555a36c6bda504671f232b6e9433e4f429a2e3fd  sparc-timer.diff
> >
> > if the only issue left is the sometimes unneeded stx instruction
> > then i approve the patch with this. If there are other issues they
> > of course have to be fixed first
> 
> Apparently the ifdef test in the patch is not reliable.
> 
> > but 1 unneeded instruction in code that is used primarely for
> > debuging and otherwise is in no way speed critical and on a rare
> > arch is not worth the work&discussion here
> 
> If two or three extra lines can make it optimal, I think that should
> be done.

iam not against it if someone wants to do it but i dont think its reasonable
to require it

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Avoid a single point of failure, be that a person or equipment.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100909/12c9bbcd/attachment.pgp>