perf-toolbox / tir / 8971542243

88%

Build Type

push

github

Committed by web-flow

Commit Message

refactor: improve core IR structure (#41)

This PR introduces two major changes:
1) Operands and regions are now directly stored in the operation
2) Context now owns all the operations. The proper way to reference operations is to acquire an allocation ID and query operation from the context when needed.

As a result of the aforementioned changes, the operation defining scheme has also changed. Instead of an attr macro, we now use a derive macro. This should make ops interface and definitions more transparent and readable. It is also now possible to completely ignore the Op DSL and define all operations manually should you need to. IR Assembly is now also specified by a derive macro. Custom assembly can be achieved by implementing `Assembly` trait manually.

The interface for `Region` and `Block` was simplified by using pimpl idiom and moving `RefCell` inside these structures. Users no longer need to write `borrow()` for each operation on the block or region.

**Caution** The `Context` struct has been made thread-safe but the operations are still not thread safe on their own.

Run Details

265 of 308 new or added lines in 16 files covered. (86.04%)

3 existing lines in 3 files now uncovered.

797 of 940 relevant lines covered (84.79%)

6.13 hits per line