https://github.com/tblah approved this pull request. I don't love that we have these member variables in OmpVisitor but this is a clear improvement to what was there before so LGTM. Thanks for the cleanup. https://github.com/llvm/llvm-project/pull/159944