perf-toolbox / tir / 8772535023

88%

Build Type

push

github

Committed by web-flow

Commit Message

refactor: rethink how operations are formed (again) (#24)

- Add Builder interface to easily construct new operations
- Get rid of Operation class, just use `Rc<RefCell<dyn Op>>`
- Allow full usage of `std::any::Any` to easily cast to traits and ops
- Add shortcuts like BlockRef to be used instead of `Rc<RefCell<...>>`

Downsides:
- Now we have to use .borrow() everywhere. Plan is to replace `Rc<RefCell<...>>` with
  a custom container, similar to `Box` in terms of API.
- Operations are still scattered across the memory, leading to worse performance. Need
  to define some arena allocation APIs and tie operation lifetime to its parent Block.

Run Details

68 of 74 new or added lines in 7 files covered. (91.89%)

27 existing lines in 4 files now uncovered.

416 of 582 relevant lines covered (71.48%)

7.13 hits per line