[llvm] 0cbb1d5 - [GlobalMerge] Use constructor to set all bits in BitVector. NFC (#124375)

via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 27 12:44:48 PST 2025


Author: Craig Topper
Date: 2025-01-27T12:44:44-08:00
New Revision: 0cbb1d5673dd76112460a109c8c2af6c32c5a16d

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

LOG: [GlobalMerge] Use constructor to set all bits in BitVector. NFC (#124375)

The constructor has an optional bool for the starting value for each
bit. Use that instead of calling set().

Added: 
    

Modified: 
    llvm/lib/CodeGen/GlobalMerge.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/CodeGen/GlobalMerge.cpp b/llvm/lib/CodeGen/GlobalMerge.cpp
index 4debdea3f6568f..7b1aa7c30e581a 100644
--- a/llvm/lib/CodeGen/GlobalMerge.cpp
+++ b/llvm/lib/CodeGen/GlobalMerge.cpp
@@ -271,8 +271,7 @@ bool GlobalMergeImpl::doMerge(SmallVectorImpl<GlobalVariable *> &Globals,
 
   // If we want to just blindly group all globals together, do so.
   if (!GlobalMergeGroupByUse || (Opt.MergeConstAggressive && isConst)) {
-    BitVector AllGlobals(Globals.size());
-    AllGlobals.set();
+    BitVector AllGlobals(Globals.size(), true);
     return doMerge(Globals, AllGlobals, M, isConst, AddrSpace);
   }
 


        


More information about the llvm-commits mailing list