libhal / SJSU-Dev2

Firmware platform written by San Jose State University for building application for embedded systems
Apache License 2.0
36 stars 63 forks source link

Add OPTIMIZE to build path #1418

Closed kammce closed 3 years ago

kammce commented 3 years ago

Adding OPTIMIZE to the build path has the sole benefit of removing the need for users to run purge everytime they want to build their application with a different optimization level. This results in a quality of life improvement due to:

  1. Reduced compile times (can reuse previously generated libraries that fit the platform and the optimization level)
  2. Improved expectations regarding debugging (compiling with 'g' but previously compiled library files were compiled with 3 making debugging an unexpected disaster between code that was highly optimized and code that is optimized for debugging).

This change does, consume more space user's disk based on the permutation of platforms and optimization levels used.

Additional Updates:

codecov[bot] commented 3 years ago

Codecov Report

Merging #1418 (4100ca4) into master (c24d40c) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1418   +/-   ##
======================================
  Coverage    52.8%   52.8%           
======================================
  Files          94      94           
  Lines        4506    4506           
  Branches      964     964           
======================================
  Hits         2376    2376           
  Misses       1626    1626           
  Partials      504     504           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update c24d40c...2dc7168. Read the comment docs.