[PATCH] D152147: [RISCV] Fold binary op into select if profitable.

Mikhail Gudim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 11 10:56:36 PDT 2023


mgudim marked 2 inline comments as done.
mgudim added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:5803
+
+  SmallVector<SDValue, 2> NewConstOps({ConstSelOp, ConstBinOp});
+  if (SelOpNo == 1)
----------------
craig.topper wrote:
> You can use a plain C array and `std::swap(NewConstOps[0], NewConstOps[1])` if `SelOpNo == 1`.
Makes sence. Done.


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

https://reviews.llvm.org/D152147



More information about the llvm-commits mailing list