[PATCH] D137106: [RISCV] Fix crash a vector add has a 4x sext and zext operand.

[Quentin Colombet via Phabricator via llvm-commits]

qcolombet added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:8459
+    if (!SExt.has_value())
+      return OrigOperand;
+
----------------
That part shouldn't be needed. The `getSource()` should do the right thing.
I believe the reason it doesn't is because the `NodeExtensionHelper` got out-of-sync with the actual operand, maybe when we commute the operand of VWADD_W?

Let me check something.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137106