[FFmpeg-devel] [PATCH] read_time() for SPARC
Thu Sep 9 23:20:08 CEST 2010
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
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel