Skip to content

(3.4) backport 19606#19607

Merged
opencv-pushbot merged 1 commit intoopencv:3.4from
alalek:backport_19606
Mar 3, 2021
Merged

(3.4) backport 19606#19607
opencv-pushbot merged 1 commit intoopencv:3.4from
alalek:backport_19606

Conversation

@alalek
Copy link
Copy Markdown
Member

@alalek alalek commented Feb 23, 2021

backport #19606 (to improve merge experience with less conflicts)

- to reduce binaries size of FFmpeg Windows wrapper
- MinGW linker doesn't support -ffunction-sections (used for FFmpeg Windows wrapper)
- move code to improve locality with its used dependencies
- move UMat::dot() to matmul.dispatch.cpp (Mat::dot() is already there)
- move UMat::inv() to lapack.cpp
- move UMat::mul() to arithm.cpp
- move UMat:eye() to matrix_operations.cpp (near setIdentity() implementation)
- move normalize(): convert_scale.cpp => norm.cpp
- move convertAndUnrollScalar(): arithm.cpp => copy.cpp
- move scalarToRawData(): array.cpp => copy.cpp
- move transpose(): matrix_operations.cpp => matrix_transform.cpp
- move flip(), rotate(): copy.cpp => matrix_transform.cpp (rotate90 uses flip and transpose)
- add 'OPENCV_CORE_EXCLUDE_C_API' CMake variable to exclude compilation of C-API functions from the core module
- matrix_wrap.cpp: add compile-time checks for CUDA/OpenGL calls
- the steps above allow to reduce FFmpeg wrapper size for ~1.5Mb (initial size of OpenCV part is about 3Mb)

backport is done to improve merge experience (less conflicts)
backport of commit: 65eb946
@alalek
Copy link
Copy Markdown
Member Author

alalek commented Mar 3, 2021

👍

@opencv-pushbot opencv-pushbot merged commit a0008de into opencv:3.4 Mar 3, 2021
@alalek alalek mentioned this pull request Mar 6, 2021
@alalek alalek mentioned this pull request Apr 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: build/install category: core port/backport done Label for maintainers. Authors of PR can ignore this

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants