Conversation
alalek
left a comment
There was a problem hiding this comment.
Did you compare performance impact?
|
I did a couple of quick measures on my iPad. I run 100 successive cv::pyrDown and measured the time in milliseconds. May not be the most accurate but it gives us an idea of the results. I tried to evaluate the speed on similar-size images. New implementation (is crashing on current implementation): The new implementation is close to the current implementation for images of similar sizes. It makes me think the alignment code is behaving properly or at least doesn't drain performance much. |
This comment has been minimized.
This comment has been minimized.
d9c6e52 to
e95f5d3
Compare
resolves #12961
This pullrequest changes
Force ARM implementation to use an unaligned int pointer.