dexonsmith accepted this revision. dexonsmith added a comment. This revision is now accepted and ready to land.
LGTM, although I'd slightly prefer the change to CompilerInstance.h be split out and committed after. ================ Comment at: clang/include/clang/Frontend/CompilerInstance.h:230-232 - std::shared_ptr<CompilerInvocation> getInvocationPtr() { + CompilerInvocation &getInvocation() { assert(Invocation && "Compiler instance has no invocation!"); - return Invocation; ---------------- It'd be nice to remove this API separately in a follow up patch; the change to CompilerInstance.h is not necessary here so it's easy to split, focusing the patch on the relevant changes, and that pattern is easier for reverting (especially in out-of-tree code). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D102482/new/ https://reviews.llvm.org/D102482 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits