<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">2016-07-01 15:26 GMT+06:00 Aleksei Sidorin <span dir="ltr"><<a href="mailto:a.sidorin@samsung.com" target="_blank">a.sidorin@samsung.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">a.sidorin updated this revision to Diff 62474.<br>
a.sidorin added a comment.<br>
<br>
Fix some issues pointed by Serge Pavlov.<br>
Serge: putting a check of FriendDecls to `IsStructurallyEquivalent()` seems like a good idea for me, but we need some additional changes for it.  `CXXRecodeDecl::getFirstDecl()` is private and `StructuralEquivalenceContext()` is located in an anonymous namespace so we cannot forward-declare it in DeclCXX.h to make it friend. We can move  `StructuralEquivalenceContext()` out of anonymous namespace, however. What do you think?<br>
<div class=""><div class="h5"><br></div></div></blockquote><div class="h5"><span style="color:rgb(34,34,34)"><br></span></div><div class="h5"><span style="color:rgb(34,34,34)">I don't think this small improvement in Importer is worth invasive changes in other components.</span></div><div class="h5"><span style="color:rgb(34,34,34)"><br></span></div><div class="h5"><span style="color:rgb(34,34,34)">Thanks,</span><br></div><div class="h5"><div style="color:rgb(34,34,34)"><div class="gmail_signature" data-smartmail="gmail_signature">--Serge<br></div></div></div><div> </div></div></div></div>