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

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:223
+    setOperationAction({ISD::UINT_TO_FP, ISD::STRICT_UINT_TO_FP},
+                       {MVT::i8, MVT::i16}, Promote);
     // We have an algorithm for SSE2, and we turn this into a 64-bit
----------------
I'm really not sure this kind of change makes the code easier to understand or manage, and I'd be surprised if there was a performance gain to be had.

I'm not against adding the ArrayRef<> wrappers but I'd be a lot more careful about where they are used.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123467