[opus] Patch cleaning up Opus x86 intrinsics configury
viswanath.puttagunta at linaro.org
Tue Mar 3 20:08:38 PST 2015
On 3 March 2015 at 21:59, Jonathan Lennox <jonathan at vidyo.com> wrote:
> My patch should be against the tip, but it’s the very recent tip,
> including some changes this past Friday (27 Feb). I mentioned in the IRC
> room a problem I discovered in creating my patch, and then later improved
> the fix Tim had made for the problem. Where do you get conflicts merging
> it to tip?
> In terms of merging, you posted your patch before I posted mine, so
> probably I should be the one on the hook to rebase after your fix goes in.
> Looking over your patch quickly, I don’t think any of my changes should be
> that difficult to merge with yours.
>> Yeah.. trivial merge issues.. shouldn't take too long to resolve.
> I haven’t studied your patch in depth yet. Does Ne10 use its own RTCD
> code, or do you use Opus’s?
>> Not using Ne10's RTCD feature at the moment.. using Opus's rtcd. libopus
RTCD for linux/armv7 will not work for aarch64. It needs to be updated (or
just disabled if system built for aarch64 since Neon support is
mandatory)... It is in my todo list..
> On Mar 3, 2015, at 5:18 PM, Viswanath Puttagunta <
> viswanath.puttagunta at linaro.org> wrote:
> > Hello Jonathan,
> > I am unable to apply your patch cleanly on tip.
> > Timothy/opus-dev,
> > This patch has some conflicts with my ARM patch that does fft
> > http://lists.xiph.org/pipermail/opus/2015-March/002904.html
> > http://lists.xiph.org/pipermail/opus/2015-March/002905.html
> > One of us probably has to rebase depending on which patch goes into opus
> > Regards,
> > Vish
> > On 1 March 2015 at 20:47, Jonathan Lennox <jonathan at vidyo.com> wrote:
> >> The attached patch cleans up Opus's x86 intrinsics configury.
> >> It:
> >> * Makes —enable-intrinsics work with clang and other non-GCC compilers
> >> * Enables RTCD for the floating-point-mode SSE code in Celt.
> >> * Disables use of RTCD in cases where the compiler targets an
> >> set by default.
> >> * Enables the SSE4.1 Silk optimizations that apply to the common parts
> >> Silk when Opus is built in floating-point mode, not just in fixed-point
> >> mode.
> >> * Enables the SSE intrinsics (with RTCD when appropriate) in the Win32
> >> build.
> >> * Fixes a case where GCC would compile SSE2 code as SSE4.1, causing a
> >> on non-SSE4.1 CPUs.
> >> * Allows configuration with compilers with non-GCC-flavor flags for
> >> architecture options.
> >> * Hopefully makes the configuration and ifdef’s easier to follow and
> >> understand.
> >> This does not yet switch —enable-intrinsics to be enabled by default on
> >> supported architectures, but I think it’d be ready to do so.
> >> Comments are welcome!
> >> —
> >> Jonathan Lennox
> >> jonathan at vidyo.com
> >> _______________________________________________
> >> opus mailing list
> >> opus at xiph.org
> >> http://lists.xiph.org/mailman/listinfo/opus
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the opus