[flang-commits] [flang] [flang][OpenMP] Support custom mappers in target update to/from clauses (PR #169673)

[Krzysztof Parzyszek via flang-commits]

================
@@ -1881,13 +1881,13 @@ void OmpVisitor::ResolveMapperModifier(parser::OmpMapper &mapper) {
       context().Say(mapper.v.source,
           "Name '%s' should be a mapper name"_err_en_US, mapper.v.source);
     else
-      mapper.v.symbol = symbol;
+      const_cast<parser::OmpMapper &>(mapper).v.symbol = symbol;
----------------
kparzysz wrote:

The cast isn't necessary.  The `symbol` member is mutable:
```
struct Name {
  std::string ToString() const { return source.ToString(); }
  CharBlock source;
  mutable semantics::Symbol *symbol{nullptr}; // filled in during semantics
};
```

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