Further optimize L3_huffman and L3_imdct36
@lieff is already working on this.
Since Aug 28, 2018.
Assessment
This issue has not been assessed yet.
Description
ARM Instructions profile:
Total executed instructions: 2164536044
L3_huffman.isra.2 685600678 31.674%
mp3d_synth 546698880 25.257%
L3_imdct36 251612240 11.624%
L3_dct3_9 176638976 8.161%
mp3d_DCT_II 165811968 7.660%
mp3d_synth_pair 61793280 2.855%
L3_antialias 48054640 2.220%
L3_change_sign 36160512 1.671%
L3_midside_stereo 27845120 1.286%
get_bits 27395265 1.266%
memset 26390774 1.219%
mp3d_scale_pcm 21970944 1.015%
__memcpy_neon 19825566 0.916%
L3_ldexp_q2 17661260 0.816%
L3_read_scalefactors 14160764 0.654%
L3_decode_scalefactors 10988038 0.508%
L3_huffman and L3_imdct36+L3_dct3_9 needs optimizations. (Vectorize two L3_dct3_9?)
- Dominant language
- C
- Stars
- 2k
- Forks
- 271
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from lieff/minimp3
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
[Bug]signed left-shift overflow in mp3dec_skip_id3v1 when parsing the APEv2 tag-size field (CWE-190)Possibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 1/5 Under an hour Newbie friendliness 85/100
-
Integer overflow in Xing/Info VBR header parsing leads to ~10TB allocation (DoS)Possibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
-
Difficulty 2/5 Half a day Newbie friendliness 30/100
-
Difficulty 3/5 1-2 days Newbie friendliness 50/100
Similar issues
-
vim.filetype.match() prefixes a URL buffer name with the cwd before passing it to filetype functionsOpen
Difficulty 2/5 1-3 hours Newbie friendliness 73/100
Maintainers usually reply within 1 day
-
compilation errorOpen
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
COVESA/dlt-daemon#925 · 1 reaction ·
-
stream_curl: retry_count never resets after a successful reconnect, so long sequential streams die after 5 dropsPossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
mpv-player/mpv#18728 ·
Maintainers usually reply within 1 day
-
followup Prio 📌
Difficulty 2/5 1-3 hours Newbie friendliness 66/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
cyrusimap/cyrus-imapd#6640 ·
Maintainers usually reply within 2 days