Alan-Jowett / dagir / 19654266598

79%

push

github

Add IR JSON (de)serialization (refs #54) (#70)

* Add expression renderer unit tests and wire expression backend

* Add SPDX header to render_expr.hpp

* Use std::transform for root expressions in render_expr.hpp

* PR feedback

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* PR feedback

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* tests: remove temporary debug instrumentation and unused variable in regression equivalence test

* cmake: skip Teddy-dependent tests on macOS (TeDDy incompatible with system Clang)

* Fix tests

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* tests: assert parsedB != nullptr after render_expr parse; remove unused casts

* tests: link nlohmann_json into dagir_tests to expose header

* feature: add IR serialization header, tests, example, and docs

* fix(example): format example after stray-expression removal

* Set minimum version

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* CMake: probe for <nlohmann/json.hpp> and use single-header fallback to avoid running upstream CMake on CI

* Format: fix clang-format violation in ir_serialization.hpp

* Lint: touch ir_graph::roots to satisfy static analyzers (cppcheck)

* Cleanup: remove unused numeric_to_index from from_json

* ir_serialization: prevent ID collisions when parsing/generated IDs

---------

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>
Co-authored-by: Alan Jowett <alan.jowett@microsoft.com>

2063 of 2818 branches covered (73.21%)

Branch coverage included in aggregate %.

139 of 159 new or added lines in 2 files covered. (87.42%)

1644 of 1856 relevant lines covered (88.58%)

5489885.75 hits per line