mboehme added a comment. After some offline discussion with reviewers, we've come to the conclusion that `AnalysisContext` isn't a sufficiently clear abstraction to warrant putting it in a header.
For the time being, I'll therefore pursue the less ambitious approach of simply eliminating the `StmtToEnvMap` interface and instead turning it into a concrete class with the implementation that used to be in `StmtToEnvMapImpl`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146507/new/ https://reviews.llvm.org/D146507 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits