[PATCH] D23536: Remove excessive padding from ImmOp and RegOp.

Alexander Shaposhnikov via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 15 17:35:51 PDT 2016


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 {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23536.68116.patch
Type: text/x-patch
Size: 603 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160816/080fe859/attachment.bin>


More information about the cfe-commits mailing list