nmigen icon indicating copy to clipboard operation
nmigen copied to clipboard

Generated Verilog should be more readable

Open whitequark opened this issue 5 years ago • 5 comments

This is solely blocked on Yosys issue yosyshq/yosys#726. It's in my queue for some time, but the threshold for merging it is fairly high (multiple days of randomized testing), so I haven't been able to push it to completion yet.

whitequark avatar Jun 11 '19 04:06 whitequark

@whitequark Would having access to a bunch of CPU resources help with the randomized testing?

mithro avatar Jun 11 '19 07:06 mithro

@mithro It's kind of a pain to set up VlogHammer in the first place. I think I can use the M-Labs machine for testing that once I have a general confidence in the correctness of the changes to that pass.

whitequark avatar Jun 11 '19 07:06 whitequark

Would having access to a bunch of CPU resources help with the randomized testing?

Actually, what kind of resources can you provide? If it's something monstrous like a 64-core machine I'd be interested. VlogHammer is embarrassingly parallel, so it pays off to use a huge number of slower cores.

whitequark avatar Jul 14 '19 03:07 whitequark

I could provide pretty much anything on the list at https://cloud.google.com/compute/docs/machine-types

Anything like the following might work?

  • n1-standard-96
  • n1-highmem-64
  • n1-highcpu-96

mithro avatar Jul 15 '19 14:07 mithro

Fixing YosysHQ/yosys#726 proved to be extraordinarily complex, so bumping this from 0.1 milestone.

whitequark avatar Jul 31 '19 04:07 whitequark