Addressed @schweitzpgi review comments

- Replace switch case with individual case statement as per enum.
- Modified test case to use tco tool for validating lowering to llvmir.

Author: SouraVX

flang/lib/Lower/OpenMP.cpp

@@ -24,13 +24,23 @@ void Fortran::lower::genOMP(
       std::get<Fortran::parser::OmpSimpleStandaloneDirective>(
           simpleStandaloneConstruct.t);
   switch (directive.v) {
-  default:
-    TODO();
-  case parser::OmpSimpleStandaloneDirective::Directive::Barrier: {
+
+  case parser::OmpSimpleStandaloneDirective::Directive::Barrier:
     absConv.getFirOpBuilder().create<mlir::omp::BarrierOp>(
         absConv.getCurrentLocation());
     break;
-  }
+  case parser::OmpSimpleStandaloneDirective::Directive::Taskwait:
+    TODO();
+  case parser::OmpSimpleStandaloneDirective::Directive::Taskyield:
+    TODO();
+  case parser::OmpSimpleStandaloneDirective::Directive::TargetEnterData:
+    TODO();
+  case parser::OmpSimpleStandaloneDirective::Directive::TargetExitData:
+    TODO();
+  case parser::OmpSimpleStandaloneDirective::Directive::TargetUpdate:
+    TODO();
+  case parser::OmpSimpleStandaloneDirective::Directive::Ordered:
+    TODO();
   }
 }
 

flang/test/Lower/omp-barrier.f90

@@ -4,8 +4,8 @@
 ! RUN:   FileCheck %s --check-prefix=FIRDialect
 ! RUN: bbc -fopenmp -emit-llvm %s -o - | \
 ! RUN:   FileCheck %s --check-prefix=LLVMIRDialect
-! RUN: bbc -fopenmp -emit-llvm %s -o - | \
-! RUN:   mlir-translate -mlir-to-llvmir - | FileCheck %s --check-prefix=LLVMIR
+! RUN: bbc -fopenmp -emit-fir %s -o - | \
+! RUN:   tco | FileCheck %s --check-prefix=LLVMIR
 
 program barrier