Skip to content

core: workaround flipHoriz() alignment issues#16139

Merged
opencv-pushbot merged 1 commit intoopencv:3.4from
alalek:core_flip_avoid_unaligned
Dec 19, 2019
Merged

core: workaround flipHoriz() alignment issues#16139
opencv-pushbot merged 1 commit intoopencv:3.4from
alalek:core_flip_avoid_unaligned

Conversation

@alalek
Copy link
Copy Markdown
Member

@alalek alalek commented Dec 12, 2019

No description provided.

@tomoaki0705
Copy link
Copy Markdown
Contributor

Nice and clean code!
However, this issue is limited to Arm v7 + uint64 combination.
I'd vote to work inside NEON universal intrinsic rather than working in flipHoriz part.
The issue became visible in flipHoriz, but could happen anywhere using 64bit integer v_load
It's a dirty hack, but how about working with v_reinterpret_as_u64 in intrin_neon.hpp?

@tomoaki0705
Copy link
Copy Markdown
Contributor

I don't mean to disturb you, but alalek, could you have a look on my PR #16152 plz ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug category: core optimization platform: arm ARM boards related issues: RPi, NVIDIA TK/TX, etc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants