[cfe-commits] r43586 - /cfe/trunk/Driver/RewriteTest.cpp
Steve Naroff
snaroff at apple.com
Wed Oct 31 16:21:30 PDT 2007
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
More information about the cfe-commits
mailing list