[cfe-commits] r156253 - /cfe/trunk/tools/libclang/IndexBody.cpp

Argyrios Kyrtzidis kyrtzidis at apple.com
Mon May 7 10:42:11 PDT 2012


On May 6, 2012, at 5:53 AM, NAKAMURA Takumi wrote:

> Argyrios and Ted,
> 
> I have reverted Argyrios' r156142.
> Please confirm it would be reasonable.

Yes, this was depending on the commit before it that got reverted.

> 
> ...Takumi
> 
> 2012/5/6 NAKAMURA Takumi <geek4civic at gmail.com>:
>> Author: chapuni
>> Date: Sun May  6 03:24:34 2012
>> New Revision: 156253
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=156253&view=rev
>> Log:
>> Revert r156142, "[libclang] Bring back BodyIndexer::TraverseCXXOperatorCallExpr which"
>> 
>> It caused test/Index/index-many-call-ops.cpp to fail in stage2 c-index-test on selfhosting i686-cygwin and x86_64-linux since r156229 (Reverting making RecursiveASTVisitor data recursive).
>> 
>> Modified:
>>    cfe/trunk/tools/libclang/IndexBody.cpp
>> 
>> Modified: cfe/trunk/tools/libclang/IndexBody.cpp
>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/libclang/IndexBody.cpp?rev=156253&r1=156252&r2=156253&view=diff
>> ==============================================================================
>> --- cfe/trunk/tools/libclang/IndexBody.cpp (original)
>> +++ cfe/trunk/tools/libclang/IndexBody.cpp Sun May  6 03:24:34 2012
>> @@ -117,12 +117,6 @@
>>     return true;
>>   }
>> 
>> -  bool TraverseCXXOperatorCallExpr(CXXOperatorCallExpr *E) {
>> -    if (E->getOperatorLoc().isInvalid())
>> -      return true; // implicit.
>> -    return base::TraverseCXXOperatorCallExpr(E);
>> -  }
>> -
>>   bool VisitDeclStmt(DeclStmt *S) {
>>     if (IndexCtx.shouldIndexFunctionLocalSymbols())
>>       IndexCtx.indexDeclGroupRef(S->getDeclGroup());
>> 
>> 
>> _______________________________________________
>> 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