[PATCH] D123467: [NFC][CodeGen] Use ArrayRef in TargetLowering functions

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 10 11:44:03 PDT 2022


nikic added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/TargetLowering.h:2270
+      setOperationAction(Op, VT, Action);
+  }
 
----------------
It shouldn't be necessary to add four overloads. An `ArrayRef<T>` also accepts a single `T`, so just having the ArrayRef + ArrayRef variant should cover all cases.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D123467/new/

https://reviews.llvm.org/D123467



More information about the llvm-commits mailing list