New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Starred by 3 users

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Nov 2017
Cc:
Components:
NextAction: ----
OS: ----
Pri: 3
Type: Enhancement



Sign in to add a comment

Memory changes in AEC

Project Member Reported by bjornv@webrtc.org, Jul 3 2014

Issue description

Feature description (please include a link to the functional spec):
For SSE2 and NEON optimizations in AEC the number of frequency elements is 65, hence one extra due to the nature of DFT. This requires optimizations to add and run "left over" C-code by the end of each for-loop. Simply extending the memory to an even power of 4 padded with dummy zeros will make the optimization code simpler without losing in speed.

Eng owner: bjornv@webrtc.org




 
Project Member

Comment 1 by andrew@webrtc.org, Jul 3 2014

Cc: andrew@webrtc.org
Nice idea. So we'd do all frequency domain operations in this dummy space, but still only use the first 65 in the inverse-FFT?
Project Member

Comment 2 by bjornv@webrtc.org, Jul 4 2014

Partly yes. Only the optimization code will use the dummy space for more efficient vector operations. The C for loops will still loop up to PART_LEN1.
It will make the code so much more cleaner and just a tiny bit more efficient actually.
Project Member

Comment 3 by andrew@webrtc.org, Jul 6 2014

Makes sense.
Project Member

Comment 4 by tina.legrand@webrtc.org, Oct 30 2014

Labels: EngTriaged IceBox
Project Member

Comment 5 by bjornv@webrtc.org, Apr 2 2015

Labels: Pri-3
Project Member

Comment 6 by bjornv@webrtc.org, May 26 2015

Cc: bjornv@webrtc.org
Project Member

Comment 7 by bjornv@webrtc.org, Jun 9 2015

Cc: -bjornv@webrtc.org henrik.lundin@webrtc.org tina.legrand@webrtc.org
Owner: ----
Status: Available
Project Member

Comment 8 by tina.legrand@webrtc.org, Nov 24 2015

Owner: tina.legrand@webrtc.org
Project Member

Comment 9 by henrik.lundin@webrtc.org, Oct 5 2016

Components: Audio
Project Member

Comment 10 by henrik.lundin@webrtc.org, Oct 5 2016

Components: -SignalProcessing
Project Member

Comment 11 by tina.legrand@webrtc.org, Nov 7 2017

Status: Archived (was: Available)
[Bulk edit] This issue hasn't been modified the last twelve months -> archiving.

If this is still a valid issue that should be open, please reopen again.

Sign in to add a comment