Skip to content

core: move inline code from mat.inl.hpp#18640

Merged
opencv-pushbot merged 1 commit intoopencv:3.4from
alalek:core_cleanup_inline_code
Oct 24, 2020
Merged

core: move inline code from mat.inl.hpp#18640
opencv-pushbot merged 1 commit intoopencv:3.4from
alalek:core_cleanup_inline_code

Conversation

@alalek
Copy link
Copy Markdown
Member

@alalek alalek commented Oct 21, 2020

  • change preserves backward binary compatibility
  • move out large / complex functions from .hpp files
  • slightly reduce compilation time / binaries size of OpenCV / User applications
Binaries size change

Configuration: GCC 9.3.1, Release -O2, WITH_IPP=OFF, WITH_VIZ=OFF

  • lib/*.so size change (17 files w/o ts.a):
before after diff bytes diff %
27 270 216 24 670 896 2 599 320 9.5%
  • opencv_interactive-calibration
before after diff bytes diff %
166 384 141864 24520 14.7%
  • opencv_test_core (GoogleTest templates are huge)
before after diff bytes diff %
16 665 136 16 391 896 273 240 1.6%

@alalek
Copy link
Copy Markdown
Member Author

alalek commented Oct 24, 2020

👍

@opencv-pushbot opencv-pushbot merged commit 0ec9463 into opencv:3.4 Oct 24, 2020
@alalek alalek mentioned this pull request Nov 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants