[PATCH] D119544: Deferred Concept Instantiation Implementation

[Erich Keane via Phabricator via cfe-commits]

erichkeane added inline comments.


================
Comment at: clang/lib/Sema/SemaConcept.cpp:507
+
+  ContextRAII SavedContext{*this, const_cast<FunctionDecl *>(FD)};
+  LocalInstantiationScope Scope(*this, true);
----------------
erichkeane wrote:
> This line was the one that fixed the variable lookup in non-templates.
By changing `FD` to `FD->GetNonClosureContext` (with the appropriate casts), I was at least able to fix the crash.  The constraint not being checked for failure however, is still a problem.


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

https://reviews.llvm.org/D119544