PartialExecuter: Changed FunctionCounter to a DenseMap

Andres Madrid Ucha · Andres Madrid Ucha · commit 0b550c3d4cd3 · 2026-01-05T15:06:54.000+01:00
The use of FunctionCounter in the code does not seem to invalidate
its iterators so using a DenseMap should be safe.
diff --git a/llvm/lib/CheerpWriter/PartialExecuter.cpp b/llvm/lib/CheerpWriter/PartialExecuter.cpp
@@ -99,7 +99,8 @@ class PartialInterpreter : public llvm::Interpreter {
 	NewAlignmentData newAlignmentData;
 	llvm::ValueMap<llvm::Constant*, int> fullyKnownCEs;
 
-	std::map<const llvm::Function*, uint32_t> functionCounters;
+	llvm::DenseMap<const llvm::Function*, uint32_t> functionCounters;
+
 	bool isGlobalVariablePartiallyExecutable(const GlobalVariable& GVar)
 	{
 		if (!GVar.hasInitializer())

Original file line number	Diff line number	Diff line change
`@@ -99,7 +99,8 @@ class PartialInterpreter : public llvm::Interpreter {`
`99`	`99`	`NewAlignmentData newAlignmentData;`
`100`	`100`	`llvm::ValueMap<llvm::Constant*, int> fullyKnownCEs;`
`101`	`101`
`102`		`- std::map<const llvm::Function*, uint32_t> functionCounters;`
	`102`	`+ llvm::DenseMap<const llvm::Function*, uint32_t> functionCounters;`
	`103`	`+`
`103`	`104`	`bool isGlobalVariablePartiallyExecutable(const GlobalVariable& GVar)`
`104`	`105`	`{`
`105`	`106`	`if (!GVar.hasInitializer())`