Does this change any behavior? (missing test case?)

On Mon, Feb 29, 2016 at 6:46 PM, Argyrios Kyrtzidis via cfe-commits <
cfe-commits@lists.llvm.org> wrote:

> Author: akirtzidis
> Date: Mon Feb 29 20:46:32 2016
> New Revision: 262290
>
> URL: http://llvm.org/viewvc/llvm-project?rev=262290&view=rev
> Log:
> [index] Fix issue where data visitation was disabled with C++ operator
> call expressions, during indexing.
>
> Modified:
>     cfe/trunk/lib/Index/IndexBody.cpp
>
> Modified: cfe/trunk/lib/Index/IndexBody.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Index/IndexBody.cpp?rev=262290&r1=262289&r2=262290&view=diff
>
> ==============================================================================
> --- cfe/trunk/lib/Index/IndexBody.cpp (original)
> +++ cfe/trunk/lib/Index/IndexBody.cpp Mon Feb 29 20:46:32 2016
> @@ -258,7 +258,7 @@ public:
>                                     DataRecursionQueue *Q = nullptr) {
>      if (E->getOperatorLoc().isInvalid())
>        return true; // implicit.
> -    return base::TraverseCXXOperatorCallExpr(E);
> +    return base::TraverseCXXOperatorCallExpr(E, Q);
>    }
>
>    bool VisitDeclStmt(DeclStmt *S) {
> @@ -325,7 +325,7 @@ public:
>
>      auto visitForm = [&](InitListExpr *Form) {
>        for (Stmt *SubStmt : Form->children()) {
> -        if (!TraverseStmt(SubStmt))
> +        if (!TraverseStmt(SubStmt, Q))
>            return false;
>        }
>        return true;
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits@lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to