Skip to content

CMake support, performance improvements#8

Open
Deamhan wants to merge 53 commits intojjwhitney:masterfrom
Deamhan:master
Open

CMake support, performance improvements#8
Deamhan wants to merge 53 commits intojjwhitney:masterfrom
Deamhan:master

Conversation

@Deamhan
Copy link
Copy Markdown

@Deamhan Deamhan commented Mar 20, 2019

There is no new features or format changes in my code changes, my main goals is performance and portability. Thus I:

  • created CMakeLists.txt that works well with Visual Studio and QtCreator (tested)
  • profile guided optimization works well on Visual Studio
  • reduced stack consumption (now all stack frames less than 4K)
  • improved "hot" functions inlining
  • improved exception safety
  • improved I/O performance
  • modernized code according current standarts
  • eliminate exceptions to improve performance and compatibility with C code

As result time of running VS2017 generated binary (x64, Release) in my test (2 files over 300 MB, --all-in-ram mode) reduced by 30% (compared with original code), with PGO result is 35%. Performance improvement on GCC (8.3.1) about 10%, but it is better than nothing.

Deamhan added 30 commits March 14, 2019 18:30
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.

1 participant