[index] Fix crash with indexing designated init expressions inside templates.
authorArgyrios Kyrtzidis <akyrtzi@gmail.com>
Wed, 3 Aug 2016 05:38:53 +0000 (05:38 +0000)
committerArgyrios Kyrtzidis <akyrtzi@gmail.com>
Wed, 3 Aug 2016 05:38:53 +0000 (05:38 +0000)
rdar://27452869

llvm-svn: 277570

clang/lib/Index/IndexBody.cpp
clang/test/Index/Core/index-source.cpp

index 4908d85..62f4e88 100644 (file)
@@ -148,7 +148,7 @@ public:
 
   bool VisitDesignatedInitExpr(DesignatedInitExpr *E) {
     for (DesignatedInitExpr::Designator &D : llvm::reverse(E->designators())) {
-      if (D.isFieldDesignator())
+      if (D.isFieldDesignator() && D.getField())
         return IndexCtx.handleReference(D.getField(), D.getFieldLoc(), Parent,
                                         ParentDC, SymbolRoleSet(), {}, E);
     }
index 75c6396..c4e1290 100644 (file)
@@ -7,3 +7,15 @@ class TemplCls {
   // CHECK: [[@LINE-1]]:3 | constructor/C++ | TemplCls | c:@ST>1#T@TemplCls@F@TemplCls#I# | <no-cgname> | Decl,RelChild | rel: 1
   // CHECK-NEXT: RelChild | TemplCls | c:@ST>1#T@TemplCls
 };
+
+template <typename T>
+class BT {
+  struct KLR {
+    int idx;
+  };
+
+  // CHECK: [[@LINE+1]]:7 | instance-method/C++ | foo |
+  KLR foo() {
+    return { .idx = 0 }; // Make sure this doesn't trigger a crash.
+  }
+};