Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Mcross/cmake node size #129

Merged
merged 4 commits into from
Feb 13, 2022
Merged

Mcross/cmake node size #129

merged 4 commits into from
Feb 13, 2022

Commits on Feb 5, 2022

  1. Add logic to generate node size header file directly in CMake by scra… 

    …ping C++ compiler error messages.
    @matt-cross
    matt-cross committed Feb 5, 2022
    Configuration menu
    Copy the full SHA
    1a868f6 View commit details
    Browse the repository at this point in the history

  2. Integrate cmake-based container node size detection logic.

    @matt-cross
    matt-cross committed Feb 5, 2022
    Configuration menu
    Copy the full SHA
    6900277 View commit details
    Browse the repository at this point in the history

Commits on Feb 9, 2022

  1. Modify get_node_sizes_of() CMake function to invoke compiler once per… 

    … type, as some compilers (notably Microsoft) will not report multiple errors in this code.
    @matt-cross
    matt-cross committed Feb 9, 2022
    Configuration menu
    Copy the full SHA
    cbcc25e View commit details
    Browse the repository at this point in the history

Commits on Feb 10, 2022

  1. Switch node size checks to use more general "try_compile()" command, … 

    …and update regex to handle "ul" suffixes on numbers.
    @matt-cross
    matt-cross committed Feb 10, 2022
    Configuration menu
    Copy the full SHA
    2c8f03e View commit details
    Browse the repository at this point in the history