topic Actually, it looks like I in Intel® Integrated Performance Primitives

ippsFIR_32fc_I -> ippsFIR_32fc

James_v_1 — Fri, 05 Apr 2013 10:03:47 GMT

Can the source and destination parameters for ippsFIR_32fc (the non-inline version) safely be the same array?

Thanks

This is what a comment in

SergeyKostrov — Fri, 05 Apr 2013 13:36:31 GMT

This is what a comment in ipps.h header says: ... // pSrcDst - pointer to the input/output vector in in-place operation // pSrc - pointer to the input vector // pDst - pointer to the output vector ...

That would be for the in

James_v_1 — Fri, 05 Apr 2013 14:00:28 GMT

That would be for the in-place version. I'm interested the non-in-place version, which doesn't specify whether pSrc and pDst can be the same thing.

>>...I'm interested the non

SergeyKostrov — Fri, 05 Apr 2013 23:21:20 GMT

>>...I'm interested the non-in-place version, which doesn't specify whether pSrc and pDst can be the same thing... I could verify with another IPP function ippsFIR_Direct_32f because it has a similar set of parameters. If you try ippsFIR_32fc without success post codes for review.

Starting from 7.1.1 you can

Igor_A_Intel — Sun, 07 Apr 2013 08:59:26 GMT

Starting from 7.1.1 you can safely use pSrc=pDst for all functions that have _I analogue, moreover - all such "_I" functions are marked as deprecated with reccomendation to use corresponding out-of-place functionality with pSrc=pDst.

4 Sergey - don't use "direct" functions - they are also marked as deprecated and most of them internaly are implemented with InitAlloc and call to a function with state - so "direct" in most cases are just wrappers - for the question "why" I have only 1 answer - historically...

regards, Igor

>>...don't use "direct"

SergeyKostrov — Sun, 07 Apr 2013 18:29:40 GMT

>>...don't use "direct" functions - they are also marked as deprecated... Thanks and I'll look for a replacement.

Quote:Igor Astakhov (Intel)

James_v_1 — Mon, 08 Apr 2013 06:41:23 GMT

Igor Astakhov (Intel) wrote:

Starting from 7.1.1 you can safely use pSrc=pDst for all functions that have _I analogue, moreover - all such "_I" functions are marked as deprecated with reccomendation to use corresponding out-of-place functionality with pSrc=pDst.

4 Sergey - don't use "direct" functions - they are also marked as deprecated and most of them internaly are implemented with InitAlloc and call to a function with state - so "direct" in most cases are just wrappers - for the question "why" I have only 1 answer - historically...

Thanks Igor. I've already replaced the simpler _I functions (add, mul, div, etc), but wasn't sure that the same applied to the FIR functions - thanks for confirming that. I'm in the process of replacing the ippsFIRDirect functions with their non-direct counterparts too.

Quote:Igor Astakhov (Intel)

James_v_1 — Tue, 07 May 2013 07:16:00 GMT

Igor Astakhov (Intel) wrote:

Starting from 7.1.1 you can safely use pSrc=pDst for all functions that have _I analogue, moreover - all such "_I" functions are marked as deprecated with reccomendation to use corresponding out-of-place functionality with pSrc=pDst.

ippsFIR_32fc(Data, Data, FrameLength, FirStateCmplx); //<- This does not work

ippsFIR_32fc_I(Data, FrameLength, FirStateCmplx); //<- This does work

It looks like a temporary array will be needed, in IPP 7.1.0 at least. I'll test on 7.1.1 and see what happens there.

Actually, it looks like I

James_v_1 — Tue, 07 May 2013 07:34:24 GMT

Actually, it looks like I might be stuck on 7.1.0. Let me look into this a bit further.

Hi James,

Igor_A_Intel — Tue, 07 May 2013 11:26:20 GMT

Hi James,

we fixed all issues with pSrc==pDst (for out-of-place functions that have in-place deprecated analogues) in 7.1.1. If you find any problems in 7.1.1 with pSrc==pDst - please submit a bug - it will be fixed ASAP.

Regards, Igor

Thanks Igor, it looks like I

James_v_1 — Tue, 07 May 2013 11:53:37 GMT

Thanks Igor, it looks like I jumped the gun. Version 7.1.1 fixes the problem.