On Thu, Jun 25, 2015 at 1:13 PM Richard Smith <[email protected]> wrote:

> ================
> Comment at: lib/Sema/SemaTemplateInstantiate.cpp:1930-1939
> @@ -1929,12 +1929,12 @@
>
>    // \brief Record the point of instantiation.
>    if (MemberSpecializationInfo *MSInfo
>          = Instantiation->getMemberSpecializationInfo()) {
>      MSInfo->setTemplateSpecializationKind(TSK);
>      MSInfo->setPointOfInstantiation(PointOfInstantiation);
>    } else if (ClassTemplateSpecializationDecl *Spec
>          = dyn_cast<ClassTemplateSpecializationDecl>(Instantiation)) {
>      Spec->setTemplateSpecializationKind(TSK);
>      Spec->setPointOfInstantiation(PointOfInstantiation);
>    }
>
> ----------------
> Is it sufficient to move this block of code to after the bail-out below?
> It seems like the problem may be that we've done some amount of the
> instantiation work (storing the TSK on the specialization) before we bail
> out, and so the AST is left in an inconsistent state.
>

Doesn't work, we still crash...


>
> http://reviews.llvm.org/D10622
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to