• Added a test case for OMP loop for merging

Added a test for OpenMP for loop fusion. Is the sample test acceptable? I can add more in the same format.

added a test. Would like to have feedback to add more.

• added test case for parallel openmp for loop merging

The tests should be in llvm-project/llvm/test/Transforms/OpenMP.

• make changes in OpenMPOpt

See the new patch

In D90103#2352453, @abidmalikwaterloo wrote:

In D90103#2352442, @jdoerfert wrote:

Adding test cases

There are still no test cases.

Yes. I have tests. I will add it to the LIT and will update the patch.

In D90103#2352442, @jdoerfert wrote:

Adding test cases

There are still no test cases.

I tried almost everything. Let me try again.

I have been trying to update my patch by following the steps on:

Yes, I am trying to figure it out why it is not loading my tests and changes

The patch now handles conditional blocks containing parallel for loops.

In D75384#1955925, @abidmalikwaterloo wrote:

In D75384#1932847, @jdoerfert wrote:

This doesn't seem to use dominance at all. How do you handle

if (a) {
#pragma omp for
for (int i = 0; i < 10; i++)
  ;
} else {
#pragma omp for
for (int i = 0; i < 10; i++)
  ;
}

if the value of a is known at the time of compilation, we will only have one "for loop". Therefore, for this specific case, the implemented technique will see only one "for loop" at the IR level. This case should be a problem.

In D75384#1932847, @jdoerfert wrote:

This doesn't seem to use dominance at all. How do you handle

if (a) {
#pragma omp for
for (int i = 0; i < 10; i++)
  ;
} else {
#pragma omp for
for (int i = 0; i < 10; i++)
  ;
}