[PATCH] D116792: [AST] lookup in parent DeclContext for transparent DeclContext

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 7 06:21:12 PST 2022


erichkeane added a comment.

I had to do something similar for this at one point: https://github.com/llvm/llvm-project/commit/90010c2e1d60c6a9a4a0b30a113d4dae2b7214eb

I seem to remember hitting this assert, and from my end, I think I decided even calling 'lookup' with the linkage spec to be a mistake (BTW, you might consider updating that 'Encloses' for 'export' as well!).

Is there any mechanism in the parent call of 'lookup' here to make it get the right thing?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116792



More information about the cfe-commits mailing list