Skip to content

Conversation

@costamag
Copy link
Contributor

@costamag costamag commented May 2, 2025

This PR introduces a new engine to perform Boolean evaluation (simulation) of XAIG index lists, along with supporting changes to improve code structure and robustness.

Changes introduced:

  1. Refactored and reorganized directories in utils for better readability.
  2. Added helper functions (is_complemented, get_pi_index, get_node_index) to xag_index_list to encapsulate literal manipulations and reduce potential for errors.
  3. Implemented a simulator for XAIG index lists that evaluates nodes in topological order and manages simulation patterns efficiently.

TODO:
Update remaining code that directly manipulates literals without using xag_index_list interface functions.

This PR introduces a new engine to perform Boolean evaluation (simulation) of XAIG index lists, along with supporting changes to improve code structure and robustness.

**Changes introduced:**

1. Refactored and reorganized directories in `utils` for better readability.
2. Added helper functions (`is_complemented`, `get_pi_index`, `get_node_index`) to `xag_index_list` to encapsulate literal manipulations and reduce potential for errors.
3. Implemented a simulator for XAIG index lists that evaluates nodes in topological order and manages simulation patterns efficiently.

**TODO:**
Update remaining code that directly manipulates literals without using xag_index_list interface functions.
@costamag costamag requested a review from changmg May 2, 2025 15:58
@costamag costamag changed the title feat: simulator for XAIG index lists Simulator for XAIG index lists May 2, 2025
@changmg
Copy link

changmg commented May 5, 2025

I did not find problems in the current changes to the index lists and the simulator.

@costamag
Copy link
Contributor Author

costamag commented May 5, 2025

Thank you @changmg, I will merge the PR.

@costamag costamag merged commit 046c2b1 into lsils:master May 5, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants