Adding test cases
Make cahnges in OpenMPOpt.cpp
abidmalikwaterloo on Oct 24 2020, 12:44 PM.Authored by
On the first look, you should at least clang-format this as it is not very readable right now.
There are still no tests.
The optimization code is lacking in various regards. Let's focus on addressing the test remarks first in order to upstream those. We can revisit how to proceed with the optimization code after, e.g., revisit the algorithm from scratch.
I have a new patch that I wrote with LLVM-11 .0. The current patchwork is based on LLVM-9.0. The OpenMPOpt.CPP has a lot of modifications. What is the best way to submit the new patch? Any suggestion would be helpful.
I would like to submit the tests along with the new patch. Some tests are failing with this patch. Or should I just submit all possible tests first ( whether the patch successfully runs or not)?
Let's focus on the tests without the optimization code.
On that front: