ChuanqiXu added a comment. In D130585#3680189 <https://reviews.llvm.org/D130585#3680189>, @ilya-biryukov wrote:
> I wonder whether the right fix for this should generalize to `Mergeable<T>`. > The fact that `Mergeable::getFirstDecl` checks if the decl is coming from the > AST file before looking up its primary merged decl is probably a performance > optimization, not a deliberate semantic. Agreed. I think we could omit the `isFromASTFile` part. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130585/new/ https://reviews.llvm.org/D130585 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
