search

parallel-runtimes / lomp

Little OpenMP Library
Apache License 2.0
153 stars 17 forks source link

Duplicate, unnecessary, file. #10

Closed JimCownie closed 3 years ago

JimCownie commented 3 years ago

Describe the bug The file stats-timing.cc is present in both the microBM and src directories. The one in microBM is old and unused, so should be removed.

To Reproduce Please provide steps to reproduce the behavior:

  1. cd top-level-dir

  2. $ find . -name stats-timing.cc ./microBM/stats-timing.cc ./src/stats-timing.cc

  3. Observe the two files, then check the CMakeLists.txt in microBM to verify that all of the executables are built with add_executable(yield yield.cc ../src/stats-timing.cc ../src/util.cc) which references the file in ../src, not the local one.

Expected behavior There should only be the source file which is needed, not the old, confusing, one in microBM

Desktop (please complete the following information):

mjklemm commented 3 years ago

This has been merged a good while ago.