[flang-commits] [flang] [llvm] [Flang][OpenMP] Initial defaultmap implementation (PR #135226)

[via flang-commits]

================
@@ -32,6 +32,10 @@ namespace Fortran {
 namespace lower {
 namespace omp {
 
+// Container type for tracking user specified Defaultmaps for a target region
+using DefaultMapsTy = std::map<clause::Defaultmap::VariableCategory,
----------------
agozillon wrote:

I think I'd prefer to keep it as a map, primarily as I think IndexMap would require me to either extend VariableCategory to have an operator overload for () that converts the stored enum to an integer as I don't think enum conversion to an integer is implicit (at least when used with indexed map) or a wrapper class that'd do the same but wrap VariableCategory, so we can avoid altering VariableCategory itself. Happy to do either of course, but seems like a little extra complexity! However, there's every chance I'm misunderstanding how IndexedMap works, it's my first time using it/tinkering with it (thank you for pointing it out, I tend to forget we have a lot of nice LLVM utilities) :-)

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