[mythtv] [PATCH] dtk libsoundtouch bug

Daniel Thor Kristjansson danielk at mrl.nyu.edu
Sat Jan 29 09:27:24 EST 2005


On Sat, 29 Jan 2005, Mark Spieth wrote:
]found it. couple of transcription errors in the old macros only. new 
]inrtinsic code worked fine. all shifts require a 64bit shift arg so 
]made them regs again to minimize mem accesses.

Thanks for hunting that down.


]also dont need the -fPIC 
]options. so I removed them too. static lib after all.

Isn't this getting statically linked into a dynamic lib?
Is it safe to get rid of -fPIC then?


]also the "new" intrisics are much slower. I had a look at the assembly
]produced and its quite inefficient at optimising the registers (gcc3.3.4).
]Id keep the older macros for now for performance reasons.

Yep. But icc and newer gcc's like them. And those compilers may even 
clobber the old intrinsics in certain situations.

-- Daniel




More information about the mythtv-dev mailing list