ChuanqiXu added a comment.

In D114908#3223981 <https://reviews.llvm.org/D114908#3223981>, @rnk wrote:

> Friendly ping for a modules CTAD bugfix.

Not sure if you met the same problem with me. In our downstream, we did a 
workaround like:

  if (Context.getLangOpts().CPlusPlusModules &&
        To->getOwningModule() != From->getOwningModule())
      return false;

We workarounded the problem by inserting the above checking in 
`inheritDefaultTemplateArgument()` in ASTReaderDecl. Hope this helps.
(I understand this is not a good solution)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114908

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D114908: ... Reid Kleckner via Phabricator via cfe-commits
    • [PATCH] D114... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D114... Arthur Eubanks via Phabricator via cfe-commits
    • [PATCH] D114... Chuanqi Xu via Phabricator via cfe-commits

Reply via email to