Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove bounds checking for table lookup #79

Merged
merged 1 commit into from Sep 27, 2019

Conversation

newhoggy
Copy link
Member

@newhoggy newhoggy commented Sep 27, 2019

Before

$ time hw-xml create-index --input psd7003.xml --ib-output psd7003.xml.ib.idx --bp-output psd7003.xml.bp.idx --method stream
hw-xml create-index --input psd7003.xml --ib-output psd7003.xml.ib.idx      31.22s user 3.83s system 219% cpu 15.978 total
$ time hw-xml create-index --input psd7003.xml --ib-output psd7003.xml.ib.idx --bp-output psd7003.xml.bp.idx --method stream
hw-xml create-index --input psd7003.xml --ib-output psd7003.xml.ib.idx      30.80s user 3.86s system 217% cpu 15.926 total
$ time hw-xml create-index --input psd7003.xml --ib-output psd7003.xml.ib.idx --bp-output psd7003.xml.bp.idx --method stream
hw-xml create-index --input psd7003.xml --ib-output psd7003.xml.ib.idx      30.12s user 3.68s system 213% cpu 15.812 total

After

$ time hw-xml create-index --input psd7003.xml --ib-output psd7003.xml.ib.idx --bp-output psd7003.xml.bp.idx --method stream
hw-xml create-index --input psd7003.xml --ib-output psd7003.xml.ib.idx      27.59s user 3.41s system 226% cpu 13.666 total
$ time hw-xml create-index --input psd7003.xml --ib-output psd7003.xml.ib.idx --bp-output psd7003.xml.bp.idx --method stream
hw-xml create-index --input psd7003.xml --ib-output psd7003.xml.ib.idx      27.46s user 3.57s system 224% cpu 13.799 total
$ time hw-xml create-index --input psd7003.xml --ib-output psd7003.xml.ib.idx --bp-output psd7003.xml.bp.idx --method stream
hw-xml create-index --input psd7003.xml --ib-output psd7003.xml.ib.idx      29.24s user 3.96s system 229% cpu 14.458 total

@newhoggy newhoggy merged commit c74ab60 into master Sep 27, 2019
@newhoggy newhoggy deleted the remove-bounds-checking-for-table-lookup branch September 27, 2019 03:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant