Skip to content

Simulator for XAIG index lists #675

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 5, 2025
Merged

Simulator for XAIG index lists #675

merged 2 commits into from
May 5, 2025

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