[PATCH] D149771: [RISCV] Use setcc to truncate results in widenVectorOpsToi8

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 3 13:26:08 PDT 2023


craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.

LGTM with that one comment.



================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:7298
+    TruncVals.push_back(
+        DAG.getSetCC(DL, N->getSimpleValueType(I), SDValue(WideN.getNode(), I),
+                     DAG.getConstant(0, DL, WideVT), ISD::SETNE));
----------------
While you're here

`SDValue(WideN.getNode(), I)` -> `WidenN.getValue(I)`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149771



More information about the llvm-commits mailing list