Skip to content

Conversation

@YanzhouTang
Copy link
Collaborator

Added support for existence: false in tile_overrides to remove tiles from the architecture.

Modified link creation logic to skip removed tiles, preventing crashes when building mesh topology.

Copy link
Contributor

@tancheng tancheng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we have a FIR test inside https://github.com/coredac/dataflow/tree/main/test, make it consume a custom_arch.yaml

// RUN: --architecture-spec=../../arch_spec/architecture.yaml \

And verify the after mapping IR has no operation mapped onto the tile that is removed.

@YanzhouTang
Copy link
Collaborator Author

Can we have a FIR test inside https://github.com/coredac/dataflow/tree/main/test, make it consume a custom_arch.yaml

// RUN: --architecture-spec=../../arch_spec/architecture.yaml \

And verify the after mapping IR has no operation mapped onto the tile that is removed.

I've added a testcase based on fir kernel, removing tile (0,1) and tile (1,1) which are used in normal fir kernel.

@tancheng tancheng merged commit 4fcbb7a into coredac:main Dec 21, 2025
1 check 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