diff options
author | Simon Pilgrim <llvm-dev@redking.me.uk> | 2024-04-19 16:56:53 +0100 |
---|---|---|
committer | Simon Pilgrim <llvm-dev@redking.me.uk> | 2024-04-19 16:57:18 +0100 |
commit | 26a59bf6fcfe3717d41ae6a92b8e0d3913c3fc1f (patch) | |
tree | 22a04d50780e200c48e5509b090af889b80c770b | |
parent | 64cc3fad53c8313ad2fb3f2ba2fd44f280e5babf (diff) | |
download | llvm-26a59bf6fcfe3717d41ae6a92b8e0d3913c3fc1f.zip llvm-26a59bf6fcfe3717d41ae6a92b8e0d3913c3fc1f.tar.gz llvm-26a59bf6fcfe3717d41ae6a92b8e0d3913c3fc1f.tar.bz2 |
[TableGen] MacroFusionPredicatorEmitter - pass constant std::vector arguments by ArrayRef instead
Silence pass by value warnings
Fixes #89210
-rw-r--r-- | llvm/utils/TableGen/MacroFusionPredicatorEmitter.cpp | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/llvm/utils/TableGen/MacroFusionPredicatorEmitter.cpp b/llvm/utils/TableGen/MacroFusionPredicatorEmitter.cpp index 63b827a..f61a058 100644 --- a/llvm/utils/TableGen/MacroFusionPredicatorEmitter.cpp +++ b/llvm/utils/TableGen/MacroFusionPredicatorEmitter.cpp @@ -55,11 +55,11 @@ class MacroFusionPredicatorEmitter { RecordKeeper &Records; CodeGenTarget Target; - void emitMacroFusionDecl(std::vector<Record *> Fusions, PredicateExpander &PE, + void emitMacroFusionDecl(ArrayRef<Record *> Fusions, PredicateExpander &PE, raw_ostream &OS); - void emitMacroFusionImpl(std::vector<Record *> Fusions, PredicateExpander &PE, + void emitMacroFusionImpl(ArrayRef<Record *> Fusions, PredicateExpander &PE, raw_ostream &OS); - void emitPredicates(std::vector<Record *> &FirstPredicate, bool IsCommutable, + void emitPredicates(ArrayRef<Record *> FirstPredicate, bool IsCommutable, PredicateExpander &PE, raw_ostream &OS); void emitFirstPredicate(Record *SecondPredicate, bool IsCommutable, PredicateExpander &PE, raw_ostream &OS); @@ -76,7 +76,7 @@ public: } // End anonymous namespace. void MacroFusionPredicatorEmitter::emitMacroFusionDecl( - std::vector<Record *> Fusions, PredicateExpander &PE, raw_ostream &OS) { + ArrayRef<Record *> Fusions, PredicateExpander &PE, raw_ostream &OS) { OS << "#ifdef GET_" << Target.getName() << "_MACRO_FUSION_PRED_DECL\n"; OS << "#undef GET_" << Target.getName() << "_MACRO_FUSION_PRED_DECL\n\n"; OS << "namespace llvm {\n"; @@ -93,7 +93,7 @@ void MacroFusionPredicatorEmitter::emitMacroFusionDecl( } void MacroFusionPredicatorEmitter::emitMacroFusionImpl( - std::vector<Record *> Fusions, PredicateExpander &PE, raw_ostream &OS) { + ArrayRef<Record *> Fusions, PredicateExpander &PE, raw_ostream &OS) { OS << "#ifdef GET_" << Target.getName() << "_MACRO_FUSION_PRED_IMPL\n"; OS << "#undef GET_" << Target.getName() << "_MACRO_FUSION_PRED_IMPL\n\n"; OS << "namespace llvm {\n"; @@ -121,9 +121,10 @@ void MacroFusionPredicatorEmitter::emitMacroFusionImpl( OS << "\n#endif\n"; } -void MacroFusionPredicatorEmitter::emitPredicates( - std::vector<Record *> &Predicates, bool IsCommutable, PredicateExpander &PE, - raw_ostream &OS) { +void MacroFusionPredicatorEmitter::emitPredicates(ArrayRef<Record *> Predicates, + bool IsCommutable, + PredicateExpander &PE, + raw_ostream &OS) { for (Record *Predicate : Predicates) { Record *Target = Predicate->getValueAsDef("Target"); if (Target->getName() == "first_fusion_target") |