[PATCH] Fix for crash due to g++.old-deja/g++.other/using3.C

Dinesh Dwivedi dinesh.d at samsung.com
Tue Apr 29 03:11:19 PDT 2014


================
Comment at: lib/Sema/SemaDeclCXX.cpp:7335-7337
@@ -7329,4 +7334,5 @@
+
     if (RequireMember && !isa<FieldDecl>(ND) && !isa<CXXMethodDecl>(ND) &&
         !isa<TypeDecl>(ND))
       return false;
 
----------------
Richard Smith wrote:
> Is this still necessary?
yes, I have added one test case which fails if we comment these lines. I have kept these lines so that my change just filter out non class member if RequiredMember is true but should not add any other type of class member which it was not considering before my changes.

http://reviews.llvm.org/D3051






More information about the cfe-commits mailing list