Re: [FFmpeg-devel] [PATCH] avcodec/rdft: remove sintable

2017-07-06 Thread Muhammad Faiz
On Wed, Jul 5, 2017 at 10:22 PM, Rostislav Pehlivanov wrote: > On 5 July 2017 at 11:33, Muhammad Faiz wrote: > >> It is redundant with costable. The first half of sintable is >> identical with the second half of costable. The second half >> of sintable is negative value of the first half of sinta

Re: [FFmpeg-devel] [PATCH] avcodec/rdft: remove sintable

2017-07-05 Thread Rostislav Pehlivanov
On 5 July 2017 at 11:33, Muhammad Faiz wrote: > It is redundant with costable. The first half of sintable is > identical with the second half of costable. The second half > of sintable is negative value of the first half of sintable. > > The computation is changed to handle sign of sin values, >

[FFmpeg-devel] [PATCH] avcodec/rdft: remove sintable

2017-07-05 Thread Muhammad Faiz
It is redundant with costable. The first half of sintable is identical with the second half of costable. The second half of sintable is negative value of the first half of sintable. The computation is changed to handle sign of sin values, so disable arm optimization. Signed-off-by: Muhammad Faiz