Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

C++17 standard + higher optimizations in Release build #82

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

aminya
Copy link
Contributor

@aminya aminya commented Dec 3, 2020

Requires #88

Description of the change

This PR adds a release configuration (as the default configuration) that:

  • Uses C++17 standard
  • Uses higher compiler optimizations

Benefits

  • Newer C++ standard allows using better algorithms, optimizations, and better defaults in general. In future PRs, we can make use of C++17 features such as built-in optional support.
  • Higher compiler optimizations instruct the compilers to work hard for optimizing the code. This results in higher performance

Verifications

The tests pass.

Release Notes

  • Uses C++17 standard
  • Uses higher compiler optimizations

@aminya aminya force-pushed the Release-optimizations branch from d73adf7 to efe74c6 Compare December 4, 2020 19:51
@aminya aminya mentioned this pull request Dec 5, 2020
@aminya aminya force-pushed the Release-optimizations branch from efe74c6 to 38128a3 Compare December 5, 2020 21:51
@aminya
Copy link
Contributor Author

aminya commented Dec 8, 2020

Travis is truly broken. I can't install clang on it (tried clang 10 and 9). I would merge #88 to fix all the CI issues instead of trying to fix Travis.

This reverts commit 833f7dd.

Update README.md
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant