https://github.com/ChuanqiXu9 commented:
LGTM. But I didn't recognize you didn't update `MultiplexExternalSemaSource`. Every time we add new interface to `ExternalASTSource`, we need to update `MultiplexExternalSemaSource` too. https://github.com/llvm/llvm-project/pull/132214 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits