[PATCH] D27970: [ThinLTO] Save 8B per summary entry by rearranging the fields (NFC)

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 21 20:20:04 PST 2016


This revision was automatically updated to reflect the committed changes.
Closed by commit rL290314: [ThinLTO] Save 8B per summary entry by rearranging the fields (NFC) (authored by mehdi_amini).

Changed prior to commit:
  https://reviews.llvm.org/D27970?vs=82062&id=82305#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D27970

Files:
  llvm/trunk/include/llvm/IR/ModuleSummaryIndex.h


Index: llvm/trunk/include/llvm/IR/ModuleSummaryIndex.h
===================================================================
--- llvm/trunk/include/llvm/IR/ModuleSummaryIndex.h
+++ llvm/trunk/include/llvm/IR/ModuleSummaryIndex.h
@@ -100,7 +100,7 @@
 class GlobalValueSummary {
 public:
   /// \brief Sububclass discriminator (for dyn_cast<> et al.)
-  enum SummaryKind { AliasKind, FunctionKind, GlobalVarKind };
+  enum SummaryKind : unsigned { AliasKind, FunctionKind, GlobalVarKind };
 
   /// Group flags (Linkage, noRename, isOptSize, etc.) as a bitfield.
   struct GVFlags {
@@ -152,6 +152,8 @@
   /// Kind of summary for use in dyn_cast<> et al.
   SummaryKind Kind;
 
+  GVFlags Flags;
+
   /// This is the hash of the name of the symbol in the original file. It is
   /// identical to the GUID for global symbols, but differs for local since the
   /// GUID includes the module level id in the hash.
@@ -166,8 +168,6 @@
   /// module path string table.
   StringRef ModulePath;
 
-  GVFlags Flags;
-
   /// List of values referenced by this global value's definition
   /// (either by the initializer of a global variable, or referenced
   /// from within a function). This does not include functions called, which


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27970.82305.patch
Type: text/x-patch
Size: 1222 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161222/639c7b0d/attachment.bin>


More information about the llvm-commits mailing list