jingpu / Halide-HLS

HLS branch of Halide
Other
77 stars 22 forks source link

More readable HLS arg name #31

Closed Kuree closed 6 years ago

Kuree commented 6 years ago

Introduced a guess_name function in HLS codegen so that instead of using arg_%d, it's now using more human readable naming used in Halide code. This is extremely helpful because user now can specify the DMA channel loading order in the driver code automatically, given the pipeline_zynq buffer preparing sequence. If the code fails to guess the argument name, it will fall back to arg_%d.

Kuree commented 6 years ago

Fixed in a889989.

Kuree commented 6 years ago

Done.

jingpu commented 6 years ago

The build failure looks relative.

Kuree commented 6 years ago

It seems like static functions cannot have const qualifier. See: https://stackoverflow.com/a/19812883

I will revert it back to non-const function.