Forum Discussion

Pan's avatar
Pan
Icon for New Contributor rankNew Contributor
6 years ago
Solved

Intel HLS (19.3) - Bad implementation of memory.

The expected implementation should have 1 Write (2 shared stores) and 1 Read (1 load) port. But those are the implementations in different cases: REG2_DATA_WIDTH 40 - Wrong implementation. The memo...
  • MEIYAN_L_Intel's avatar
    MEIYAN_L_Intel
    6 years ago

    Hi,

    I had tried to compile the code and the attached code is not able to compile.

    I have added the following line to make it compile:

    #include <HLS/ac_int.h>

    #include <HLS/hls.h>

    #define HLS_COMPONENT component

    #define REG_ADDR WIDTH 64

    This appears to be a regression from 19.1; when compiled using 19.1, the 40-bit and 41-bit both worked nicely. It appears that in the 40-bit case, the compiler decomposes the 40-bit store into power-of-2 stores, 8+8+8+16. A workaround is to set the type in memory to be a 64-bit ac_int.

    Thanks

MEIYAN_L_Intel's avatar
MEIYAN_L_Intel
Icon for Frequent Contributor rankFrequent Contributor
6 years ago

Hi,

I had reported this problem to the developer and they had noted this problem as well.

They plan to solved this problem in newer version of Intel HLS.

Thank you for your information.

Thanks