DLA: mergeNodes now uses ArrayRef argument

This allows for a greater flexibility in its users.
This commit is contained in:
Pietro Fezzardi
2023-08-17 12:06:46 +02:00
parent cac7fee07d
commit 4e32f05aba
2 changed files with 9 additions and 4 deletions
+7 -3
View File
@@ -5,6 +5,7 @@
#include <algorithm>
#include <string>
#include "llvm/ADT/ArrayRef.h"
#include "llvm/ADT/SCCIterator.h"
#include "llvm/ADT/STLExtras.h"
#include "llvm/ADT/SmallString.h"
@@ -281,10 +282,13 @@ static void fixPredSucc(LayoutTypeSystemNode *From,
static Logger<> MergeLog("dla-merge-nodes");
using LayoutTypeSystemNodePtrVec = std::vector<LayoutTypeSystemNode *>;
void LayoutTypeSystem::mergeNodes(llvm::ArrayRef<LayoutTypeSystemNode *>
ToMerge) {
revng_assert(ToMerge.size() > 0ULL);
// If we're merging just one node there's nothing to do.
if (ToMerge.size() <= 1ULL)
return;
void LayoutTypeSystem::mergeNodes(const LayoutTypeSystemNodePtrVec &ToMerge) {
revng_assert(ToMerge.size() > 1ULL);
LayoutTypeSystemNode *Into = ToMerge[0];
const unsigned IntoID = Into->ID;