Skip to content
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

vtr arch.xml file does not adapt to at least some non-default COFFE settings #30

Open
vaughnbetz opened this issue Jul 28, 2020 · 0 comments

Comments

@vaughnbetz
Copy link
Owner

vaughnbetz commented Jul 28, 2020

The vtr arch.xml file that COFFE can output does not adapt in some non-delay/non-area ways to the input. The number of inputs and the number of BLEs in particular do not affect the generated arch.xml (beyond affecting the area and delay, which are correct). The attached file shows a case with a CLB of eight 4-LUTs and 18 inputs, but you can see the clb in the architecture xml file has 10 LUTs and 32 inputs. The area and delay values match the COFFE log file, which shows it did size the correct clb; the arch.xml writing is just too hard-coded.
coffe_input_flut08-4.txtcoffe_log_8_fracturable4LUT.log
vtr_arch_flut08-4-xml.txt

This should be fixed -- the vtr.arch file should reflect all the parameters sent into COFFE. If that is too hard in some cases, we should at least print a warning and a banner saying where to fix up the vtr.arch file.

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

No branches or pull requests

1 participant