myatsina added a comment.

In http://reviews.llvm.org/D18123#374170, @hintonda wrote:

> LookupResult's copy ctor and assignment operator are used in 
> Sema::LookupInlineAsmField().  Looks like these were added back in December.
>
> I'll remove the defaults, from this patch, but not sure how to handle 
> LookupInlineAsmField().  Will add author of that change.
>
> OTOH, UnresolvedSetImpl requires an assignment operator since it's a base to 
> UnresolvedSet which has an implicit assignment operator, so I think that 
> change is correct.


I can change my code to avoid the copy ctor and assignment operator of 
LookupResult - In every iteration I only need to access 2 members of the 
previous LookupResult (isSingleResult() and getFoundDecl()), I can keep only 
them instead of the whole object.
Do you want me to upload a patch with this change?


http://reviews.llvm.org/D18123



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to