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

[Sergio Afonso 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,
----------------
skatrak wrote:

I think `IndexedMap` would be a better choice according to the [LLVM Programmer's Manual](https://llvm.org/docs/ProgrammersManual.html#llvm-adt-indexedmap-h), though I'm not sure if there's a good `null` value to use for `clause::Defaultmap::VariableCategory`.
```suggestion
using DefaultMapsTy = llvm::IndexedMap<clause::Defaultmap::VariableCategory,
```

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