[llvm] d5d3f98 - [LTO] Modernize AliasSummary and ModuleSummaryIndex (NFC) (#107633)

via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 6 15:56:28 PDT 2024


Author: Kazu Hirata
Date: 2024-09-06T15:56:23-07:00
New Revision: d5d3f989689978850b23e6a3ab4520fb6e4f8b52

URL: https://github.com/llvm/llvm-project/commit/d5d3f989689978850b23e6a3ab4520fb6e4f8b52
DIFF: https://github.com/llvm/llvm-project/commit/d5d3f989689978850b23e6a3ab4520fb6e4f8b52.diff

LOG: [LTO] Modernize AliasSummary and ModuleSummaryIndex (NFC) (#107633)

Added: 
    

Modified: 
    llvm/include/llvm/IR/ModuleSummaryIndex.h

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/IR/ModuleSummaryIndex.h b/llvm/include/llvm/IR/ModuleSummaryIndex.h
index 7bd90d3ed7e9d7..e4d188dc22d7c5 100644
--- a/llvm/include/llvm/IR/ModuleSummaryIndex.h
+++ b/llvm/include/llvm/IR/ModuleSummaryIndex.h
@@ -640,12 +640,11 @@ class AliasSummary : public GlobalValueSummary {
   /// memory for time). Note that this pointer may be null (and the value info
   /// empty) when we have a distributed index where the alias is being imported
   /// (as a copy of the aliasee), but the aliasee is not.
-  GlobalValueSummary *AliaseeSummary;
+  GlobalValueSummary *AliaseeSummary = nullptr;
 
 public:
   AliasSummary(GVFlags Flags)
-      : GlobalValueSummary(AliasKind, Flags, SmallVector<ValueInfo, 0>{}),
-        AliaseeSummary(nullptr) {}
+      : GlobalValueSummary(AliasKind, Flags, SmallVector<ValueInfo, 0>{}) {}
 
   /// Check if this is an alias summary.
   static bool classof(const GlobalValueSummary *GVS) {
@@ -1420,7 +1419,7 @@ class ModuleSummaryIndex {
   // used except in the case of a SamplePGO partial profile, and should be
   // reevaluated/redesigned to allow more effective incremental builds in that
   // case.
-  uint64_t BlockCount;
+  uint64_t BlockCount = 0;
 
   // List of unique stack ids (hashes). We use a 4B index of the id in the
   // stack id lists on the alloc and callsite summaries for memory savings,
@@ -1446,7 +1445,7 @@ class ModuleSummaryIndex {
   ModuleSummaryIndex(bool HaveGVs, bool EnableSplitLTOUnit = false,
                      bool UnifiedLTO = false)
       : HaveGVs(HaveGVs), EnableSplitLTOUnit(EnableSplitLTOUnit),
-        UnifiedLTO(UnifiedLTO), Saver(Alloc), BlockCount(0) {}
+        UnifiedLTO(UnifiedLTO), Saver(Alloc) {}
 
   // Current version for the module summary in bitcode files.
   // The BitcodeSummaryVersion should be bumped whenever we introduce changes


        


More information about the llvm-commits mailing list