There's no functional change. This was a mistake I made when splitting the original patch into pieces and I wanted to correct before proceeding with additional changes.
On Mon, Feb 27, 2017 at 8:27 AM, David Blaikie <dblai...@gmail.com> wrote: > An explanation as to why the code was moved (& possibly test cases, or > "NFC" in the description) would be handy here. > > On Fri, Feb 24, 2017 at 5:41 PM Richard Trieu via cfe-commits < > cfe-commits@lists.llvm.org> wrote: > >> Author: rtrieu >> Date: Fri Feb 24 19:29:34 2017 >> New Revision: 296221 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=296221&view=rev >> Log: >> [ODRHash] Move inherited visitor call to end of function. >> >> Modified: >> cfe/trunk/lib/AST/ODRHash.cpp >> >> Modified: cfe/trunk/lib/AST/ODRHash.cpp >> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/ >> ODRHash.cpp?rev=296221&r1=296220&r2=296221&view=diff >> ============================================================ >> ================== >> --- cfe/trunk/lib/AST/ODRHash.cpp (original) >> +++ cfe/trunk/lib/AST/ODRHash.cpp Fri Feb 24 19:29:34 2017 >> @@ -182,8 +182,6 @@ public: >> } >> >> void VisitFieldDecl(const FieldDecl *D) { >> - Inherited::VisitFieldDecl(D); >> - >> const bool IsBitfield = D->isBitField(); >> Hash.AddBoolean(IsBitfield); >> >> @@ -193,6 +191,8 @@ public: >> >> Hash.AddBoolean(D->isMutable()); >> AddStmt(D->getInClassInitializer()); >> + >> + Inherited::VisitFieldDecl(D); >> } >> }; >> >> >> >> _______________________________________________ >> 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
