Skip to content

[CI] Add FTD to CI pipeline - #658

Open
Jiahui17 wants to merge 6 commits into
mainfrom
add-ftd-to-ci
Open

[CI] Add FTD to CI pipeline#658
Jiahui17 wants to merge 6 commits into
mainfrom
add-ftd-to-ci

Conversation

@Jiahui17

Copy link
Copy Markdown
Member

No description provided.

@Jiahui17

Copy link
Copy Markdown
Member Author

@AyaElAkhras @QinYuan2000 I wanted to add Matvec with FTD as well, but it doesn't work in the current main. Could you please take a look?

@Jiahui17 Jiahui17 closed this Nov 27, 2025
@AyaElAkhras

Copy link
Copy Markdown
Member

Thanks @Jiahui17 for reporting! I checked from my side and the current main crashes because of some wrong casting. I need some time to identify the recent commit to the main that resulted in this.

@AyaElAkhras

Copy link
Copy Markdown
Member

This is the commit 1567b7c that results in this error. I will dig further into the commit to figure out the issue.

@AyaElAkhras AyaElAkhras reopened this Dec 5, 2025
@Jiahui17 Jiahui17 changed the title [CI] Add 3 benchmarks that uses FTD to CI pipeline [CI] Add FTD to CI pipeline Feb 2, 2026
@Jiahui17

Copy link
Copy Markdown
Member Author

@AyaElAkhras Any progress on this?

@AyaElAkhras

Copy link
Copy Markdown
Member

@AyaElAkhras Any progress on this?
We are currently working on the fix I was mentioning here #666, but we have not fully resolved all MLIR issues yet.

@Jiahui17
Jiahui17 force-pushed the add-ftd-to-ci branch 3 times, most recently from 21478f6 to d7e6a14 Compare August 19, 2026 11:03
@AyaElAkhras

Copy link
Copy Markdown
Member

Thanks, @Jiahui17, for bringing this back to life! Could you also add the --straight-to-queue flag? It goes hand in hand with FTD.

The flag essentially removes the network of CMerges by implementing the approach from my FPGA'23 paper: if there is an LSQ, it replaces the CMerge network accordingly; if the function is void, it leaves the network unchanged; and if the function is non-void, it removes the network entirely.

@Jiahui17

Copy link
Copy Markdown
Member Author

Thanks, @Jiahui17, for bringing this back to life! Could you also add the --straight-to-queue flag? It goes hand in hand with FTD.

The flag essentially removes the network of CMerges by implementing the approach from my FPGA'23 paper: if there is an LSQ, it replaces the CMerge network accordingly; if the function is void, it leaves the network unchanged; and if the function is non-void, it removes the network entirely.

does it make sense to always switch that flag on if we have --fast-token-delivery?

@AyaElAkhras

Copy link
Copy Markdown
Member

Thanks, @Jiahui17, for bringing this back to life! Could you also add the --straight-to-queue flag? It goes hand in hand with FTD.
The flag essentially removes the network of CMerges by implementing the approach from my FPGA'23 paper: if there is an LSQ, it replaces the CMerge network accordingly; if the function is void, it leaves the network unchanged; and if the function is non-void, it removes the network entirely.

does it make sense to always switch that flag on if we have --fast-token-delivery?

Yes, this works, thanks!

@Jiahui17

Jiahui17 commented Aug 19, 2026

Copy link
Copy Markdown
Member Author

Could you make sure that the new added units also work in verilog?

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