[PATCH] D113709: Don't consider `LinkageSpec` when calculating DeclContext `Encloses`

2021-11-29 Thread Erich Keane via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG90010c2e1d60: Don't consider 'LinkageSpec' when calculating DeclContext 'Encloses' (authored by erichkeane). Herald added a project: clang. Changed prior to commit: https://reviews.llvm.org/D113709?vs=3

[PATCH] D113709: Don't consider `LinkageSpec` when calculating DeclContext `Encloses`

2021-11-18 Thread John McCall via Phabricator via cfe-commits
rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. > We don't properly handle lookup through using declarations when there is a > linkage spec in the common chain. Pedantic note: you mean using *directives*. At some point, we should proba

[PATCH] D113709: Don't consider `LinkageSpec` when calculating DeclContext `Encloses`

2021-11-11 Thread Erich Keane via Phabricator via cfe-commits
erichkeane created this revision. erichkeane added reviewers: hubert.reinterpretcast, rsmith, aaron.ballman, rjmccall. erichkeane requested review of this revision. We don't properly handle lookup through using declarations when there is a linkage spec in the common chain. This is because `CppLo