-
Notifications
You must be signed in to change notification settings - Fork 431
Description
Expected Behaviour
According to the latest documentation on the VTR benchmark, there should be pre-synthetized designs at $VTR_ROOT/vtr_flow/benchmarks/vtr_benchmarks_blif
Current Behaviour
The directory is not present at the current master (b913727) and 8.x (fd69801) branches revision.
Possible Solution
- Fix the documentation by removing entry
Context
I was doing the exact same task to save time on routing simulations and got surprised when reviewing the documentation to know the resources were supposed to be available at the start. Through some quick research it seems the directory was removed 560eb92 in favor of using architecture features such as DSP blocks or hard multipliers, which makes absolute sense.
Although not critical, missleading documentation can be harsh specially for beginner users. If it makes sense to fix it, if have already prepared a pull-request to be opened. I have never actually contributed to an OSS project to be able to tell what counts as relevant PRs.
P.S. there are similar old/deprecated info about example tasks, I could also look at it again and open another issue or complement this one
P.P.S. there is a single typo at the VPR documentation that also nerves me out and I have fixed at my fork, if it also makes sense I can open the pull-request here as well.