[PATCH] D130585: [Sema] Return primary merged decl as canonical for concepts

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 26 10:17:36 PDT 2022


ilya-biryukov added a comment.

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.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130585/new/

https://reviews.llvm.org/D130585



More information about the cfe-commits mailing list