search

aras-p / ClangBuildAnalyzer

Clang build analysis tool using -ftime-trace
The Unlicense
1.02k stars 65 forks source link

Modernize CMakeLists.txt #35

Closed Lectem closed 4 years ago

Lectem commented 4 years ago

This version of CMakeLists.txt follows the "modern cmake" way by setting properties directly on the target.

I also removed the -O2 from the flags as one should use the Release or RelWithDebInfo configurations to have optimized builds. This will also make it work across various compilers.

It will also use the correct flags for c++14 using any compiler.

aras-p commented 4 years ago

I know nothing about cmake, so I'll just assume this is all fine :)

mathbunnyru commented 4 years ago

Don't worry, the change is good :)