Minor issue in loadstore_trace.cpp

CMU-SAFARI / ramulator2

Ramulator 2.0 is a modern, modular, extensible, and fast cycle-accurate DRAM simulator. It provides support for agile implementation and evaluation of new memory system designs (e.g., new DRAM standards, emerging RowHammer mitigation techniques). Described in our paper https://people.inf.ethz.ch/omutlu/pub/Ramulator2_arxiv23.pdf

https://arxiv.org/abs/2308.11030

MIT License

252 stars 62 forks source link

Minor issue in loadstore_trace.cpp #10

Closed wogus20002 closed 1 year ago

wogus20002 commented 1 year ago

https://github.com/CMU-SAFARI/ramulator2/blob/d56fd74841a2c1c5d26002a1ba6b8ca6d2d64e9d/src/frontend/impl/memory_trace/loadstore_trace.cpp#L43

It seems that Read and Write should be swapped in this line.

RichardLuo79 commented 1 year ago

Hi Jaehyun,

Thanks for spotting this issue! We have fixed this in the latest commit.