[PATCH] D24754: [cleanup] Remove excessive padding from RedeclarableResult

Vedant Kumar via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 21 16:58:55 PDT 2016


vsk added inline comments.

================
Comment at: lib/Serialization/ASTReaderDecl.cpp:154
@@ -153,3 +153,3 @@
     public:
-      RedeclarableResult(GlobalDeclID FirstID, Decl *MergeWith, bool IsKeyDecl)
-          : FirstID(FirstID), MergeWith(MergeWith), IsKeyDecl(IsKeyDecl) {}
+      RedeclarableResult(Decl *MergeWith, GlobalDeclID FirstID, bool IsKeyDecl)
+          : MergeWith(MergeWith), FirstID(FirstID), IsKeyDecl(IsKeyDecl) {}
----------------
Why do you need to change the order of the parameters in the constructor?


Repository:
  rL LLVM

https://reviews.llvm.org/D24754





More information about the cfe-commits mailing list