[llvm] 9191878 - [PowerPC] Use `SmallSetVector` to enable deterministic build (#89875)

via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 24 18:27:49 PDT 2024


Author: Kai Luo
Date: 2024-04-25T09:27:45+08:00
New Revision: 919187871d142496d53bca8732677e0811b32409

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

LOG: [PowerPC] Use `SmallSetVector` to enable deterministic build (#89875)

This is to address the issue in
https://lab.llvm.org/buildbot/#/builders/54.
`SmallSetVector` interfaces as set and can be iterated in a
deterministic order.

Added: 
    

Modified: 
    llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp b/llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp
index 6e1002c45d81cb..51b79dc2b04b4e 100644
--- a/llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp
+++ b/llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp
@@ -27,7 +27,7 @@
 #include "PPCTargetStreamer.h"
 #include "TargetInfo/PowerPCTargetInfo.h"
 #include "llvm/ADT/MapVector.h"
-#include "llvm/ADT/SmallPtrSet.h"
+#include "llvm/ADT/SetVector.h"
 #include "llvm/ADT/Statistic.h"
 #include "llvm/ADT/StringExtras.h"
 #include "llvm/ADT/StringRef.h"
@@ -242,7 +242,7 @@ class PPCAIXAsmPrinter : public PPCAsmPrinter {
 private:
   /// Symbols lowered from ExternalSymbolSDNodes, we will need to emit extern
   /// linkage for them in AIX.
-  SmallPtrSet<MCSymbol *, 8> ExtSymSDNodeSymbols;
+  SmallSetVector<MCSymbol *, 8> ExtSymSDNodeSymbols;
 
   /// A format indicator and unique trailing identifier to form part of the
   /// sinit/sterm function names.


        


More information about the llvm-commits mailing list