celerity / celerity-runtime

High-level C++ for Accelerator Clusters
https://celerity.github.io
MIT License
139 stars 18 forks source link

Fix & test pretty-printing 0-dimensional region maps #251

Closed fknorr closed 6 months ago

fknorr commented 6 months ago

region_map_impl is specialized on the 0-dimensional case but the GDB pretty printers weren't, causing them to try and access non-existing members. This PR adds the necessary pretty-printer code and adds a test.

github-actions[bot] commented 6 months ago

Check-perf-impact results: (4c65f1399a47e0eb1340f63004745b17)

:question: No new benchmark data submitted. :question:
Please re-run the microbenchmarks and include the results if your commit could potentially affect performance.

coveralls commented 6 months ago

Pull Request Test Coverage Report for Build 7887220924

Details


Files with Coverage Reduction New Missed Lines %
src/grid.cc 1 97.29%
<!-- Total: 1 -->
Totals Coverage Status
Change from base Build 7875546105: -0.04%
Covered Lines: 4998
Relevant Lines: 5194

💛 - Coveralls