[cfe-commits] r43586 - /cfe/trunk/Driver/RewriteTest.cpp

[Steve Naroff]

Interesting...that's in the code we were just discussing (that I  
didn't understand).

I knew there was a bug in that vicinity...

snaroff

On Oct 31, 2007, at 4:08 PM, Fariborz Jahanian wrote:

> Author: fjahanian
> Date: Wed Oct 31 18:08:24 2007
> New Revision: 43586
>
> URL: http://llvm.org/viewvc/llvm-project?rev=43586&view=rev
> Log:
> Fixed a bug exposed by fixing the assert in previous patch (one of  
> the tests asserted).
>
> Modified:
>     cfe/trunk/Driver/RewriteTest.cpp
>
> Modified: cfe/trunk/Driver/RewriteTest.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/Driver/ 
> RewriteTest.cpp?rev=43586&r1=43585&r2=43586&view=diff
>
> ====================================================================== 
> ========
> --- cfe/trunk/Driver/RewriteTest.cpp (original)
> +++ cfe/trunk/Driver/RewriteTest.cpp Wed Oct 31 18:08:24 2007
> @@ -544,6 +544,9 @@
>                                                 std::string &Result) {
>    assert(CDecl && "Class missing in SynthesizeObjcInternalStruct");
>    assert(CDecl->getName() && "Name missing in  
> SynthesizeObjcInternalStruct");
> +  // Do not synthesize more than once.
> +  if (ObjcSynthesizedStructs.count(CDecl))
> +    return;
>    ObjcInterfaceDecl *RCDecl = CDecl->getSuperClass();
>    if (RCDecl && !ObjcSynthesizedStructs.count(RCDecl)) {
>      // Do it for the root
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits