[llvm-branch-commits] [flang] [llvm] [mlir] [Flang][OpenMP][MLIR] Initial declare target to for variables implementation (PR #119589)

[Akash Banerjee via llvm-branch-commits]

================
@@ -3664,6 +3690,30 @@ struct MapInfoData : MapInfosTy {
     MapInfosTy::append(CurInfo);
   }
 };
+
+enum class TargetDirective : uint32_t {
+  None = 0,
+  Target = 1,
+  TargetData = 2,
+  TargetEnterData = 3,
+  TargetExitData = 4,
+  TargetUpdate = 5
+};
----------------
TIFitis wrote:

Nit: Rename to something like `TargetDirectiveTy` or `TargetDirectiveEnumTy` for clarity.

Also, should this be enclosed in a namespace?

https://github.com/llvm/llvm-project/pull/119589