Skip to content

[gen] Relax insert edge restrict#1742

Open
ShaleXIONG wants to merge 3 commits intoherd:masterfrom
ShaleXIONG:relax-insert-edge-restrict
Open

[gen] Relax insert edge restrict#1742
ShaleXIONG wants to merge 3 commits intoherd:masterfrom
ShaleXIONG:relax-insert-edge-restrict

Conversation

@ShaleXIONG
Copy link
Copy Markdown
Collaborator

@ShaleXIONG ShaleXIONG commented Mar 5, 2026

This pull relaxes the restrict to the insert edges such as ISB.

  • Remove the restrict in cycle.ml when insert edges must be adjacent by an external edge, for all diy* tools. This allowed cycle like PodWR ISB PosRR Fre PodWR Fre. Previously no tests will be generated with a warning on the ISB insert edge.
  • Remove internal filter restriction. Given two composite relaxations, $R_1$ and $R_2$, the filter now only check the last NON-insert edge in $R_1$ against the first NON-insert edge in $R_2$. In the case, those edges does not exists, it will check the first edge in $R_1$ and last edges in $R_2$. The latter behaviour derives from previous existing behaviour. For example if we have a safe or relax edge [DpAddr ISB], the filter will ignore the ISB but use DpAddr in the filtering process.

@ShaleXIONG ShaleXIONG changed the title [WIP][gen] Relax insert edge restrict [gen] Relax insert edge restrict Mar 11, 2026
@ShaleXIONG ShaleXIONG marked this pull request as ready for review March 11, 2026 11:46
@ShaleXIONG ShaleXIONG requested a review from relokin March 11, 2026 11:51
@ShaleXIONG ShaleXIONG force-pushed the relax-insert-edge-restrict branch 2 times, most recently from a5c1d94 to 01c080a Compare March 11, 2026 14:55
@ShaleXIONG ShaleXIONG force-pushed the relax-insert-edge-restrict branch from 01c080a to 82d0893 Compare March 17, 2026 09:57
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.

1 participant