[PATCH] [PATCH] Fix to PR15826 - clang hits assert in clang::ASTContext::getASTRecordLayout

Rafael EspĂ­ndola rafael.espindola at gmail.com
Mon May 6 07:37:28 PDT 2013


Please include the testcase in the patch.

On 6 May 2013 09:56, Serge Pavlov <sepavloff at gmail.com> wrote:
> Hi doug.gregor,
>
> Don't pass declaration definition to code generator if the declaration is invalid.
>
> http://llvm-reviews.chandlerc.com/D749
>
> Files:
>   lib/Sema/SemaDecl.cpp
>
> Index: lib/Sema/SemaDecl.cpp
> ===================================================================
> --- lib/Sema/SemaDecl.cpp
> +++ lib/Sema/SemaDecl.cpp
> @@ -10295,7 +10295,8 @@
>      Tag->setTopLevelDeclInObjCContainer();
>
>    // Notify the consumer that we've defined a tag.
> -  Consumer.HandleTagDeclDefinition(Tag);
> +  if (!Tag->isInvalidDecl())
> +    Consumer.HandleTagDeclDefinition(Tag);
>  }
>
>  void Sema::ActOnObjCContainerFinishDefinition() {
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>



More information about the cfe-commits mailing list