clockwork icon indicating copy to clipboard operation
clockwork copied to clipboard

In prog: harris_sch4_1pp3c compute unit: hcompute_cim_stencil has a mismatch between C++ and coreir

Open dillonhuff opened this issue 5 years ago • 2 comments

@jeffsetter @rdaly525 I'm guessing this is related to #108 but there is an error in one of the Harris sch4 compute units:

ERROR in compute unit: hcompute_cim_stencil
in0_lgxx_stencil[0] -> 18547
in0_lgxx_stencil[0] -> 0100100001110011
in1_lgxy_stencil[0] -> 56401
in1_lgxy_stencil[0] -> 1101110001010001
in2_lgyy_stencil[0] -> 23807
in2_lgyy_stencil[0] -> 0101110011111111
	coreir_result: 18585
	cpp_result   : 22681

Could you take a look at this?

Testbench is here: https://github.com/dillonhuff/clockwork/tree/master/compute_unit_regressions/harris_sch4_1pp3c/hcompute_cim_stencil

dillonhuff avatar Nov 13 '20 03:11 dillonhuff

Can confirm. After modifying "wire" to "wire signed", it passes.

jeffsetter avatar Nov 13 '20 04:11 jeffsetter

Sorry, just saw this. Ill take a look.

rdaly525 avatar Nov 17 '20 22:11 rdaly525