[llvm] [NFC][TableGen] Code cleanup in Record.h/cpp (PR #138876)

Sergei Barannikov via llvm-commits llvm-commits at lists.llvm.org
Thu May 8 13:13:58 PDT 2025


================
@@ -1317,13 +1308,11 @@ const Init *BinOpInit::Fold(const Record *CurRec) const {
 
       SmallVector<const Init *, 8> Args;
       SmallVector<const StringInit *, 8> ArgNames;
-      for (unsigned i = 0, e = LHSs->getNumArgs(); i != e; ++i) {
-        Args.push_back(LHSs->getArg(i));
-        ArgNames.push_back(LHSs->getArgName(i));
-      }
-      for (unsigned i = 0, e = RHSs->getNumArgs(); i != e; ++i) {
-        Args.push_back(RHSs->getArg(i));
-        ArgNames.push_back(RHSs->getArgName(i));
+      for (const DagInit *Dag : {LHSs, RHSs}) {
+        for (auto [Arg, ArgName] : Dag->getArgAndNames()) {
+          Args.push_back(Arg);
+          ArgNames.push_back(ArgName);
+        }
----------------
s-barannikov wrote:

Probably could be simplified to
```suggestion
        Args.append_range(Dag->getArgs());
        ArgNames.append_range(Dag->getArgNames());
```


https://github.com/llvm/llvm-project/pull/138876


More information about the llvm-commits mailing list