alexshap created this revision.
alexshap added a reviewer: tstellarAMD.
alexshap added a subscriber: cfe-commits.
alexshap changed the visibility of this Differential Revision from "Public (No 
Login Required)" to "All Users".
Herald added subscribers: arsenm, aemerson.

The structs ImmOp and RegOp are in AArch64AsmParser.cpp (inside anonymous 
namespace).
This diff changes the order of fields and removes the excessive padding (8 
bytes).

https://reviews.llvm.org/D23536

Files:
  lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp

Index: lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
===================================================================
--- lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
+++ lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
@@ -136,18 +136,18 @@
   };
 
   struct ImmOp {
+    int64_t Val;
+    ImmTy Type;
     bool IsFPImm;
-    ImmTy Type;
-    int64_t Val;
     Modifiers Mods;
   };
 
   struct RegOp {
-    unsigned RegNo;
-    Modifiers Mods;
     const MCRegisterInfo *TRI;
     const MCSubtargetInfo *STI;
+    unsigned RegNo;
     bool IsForcedVOP3;
+    Modifiers Mods;
   };
 
   union {


Index: lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
===================================================================
--- lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
+++ lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp
@@ -136,18 +136,18 @@
   };
 
   struct ImmOp {
+    int64_t Val;
+    ImmTy Type;
     bool IsFPImm;
-    ImmTy Type;
-    int64_t Val;
     Modifiers Mods;
   };
 
   struct RegOp {
-    unsigned RegNo;
-    Modifiers Mods;
     const MCRegisterInfo *TRI;
     const MCSubtargetInfo *STI;
+    unsigned RegNo;
     bool IsForcedVOP3;
+    Modifiers Mods;
   };
 
   union {
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to