[flang-commits] [flang] [flang][OpenMP] Keep hermetic embedded module parse tree alive (PR #207491)

[via flang-commits]

https://github.com/jeanPerier approved this pull request.

Thanks for the fix! This is likely also an issue with parametrized derived types outside of the OpenMP context since the related symbols store parser::Expr*.

Out of curiosity, have you noticed/measured a noticeable increase in the memory usage of flang with this patch when compiling the application where you saw the bug with hermetic modules?

The alternative is to update the symbols to not carry pointers to the parse tree, but that would lead to duplicating representation logic since parse tree node cannot be copied.

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