[PATCH] D59119: [SelectionDAG] Check legality for ADDCARRY in expandMUL_LOHI

[Leonard Chan via Phabricator via llvm-commits]

leonardchan added a comment.

In D59119#1486753 <https://reviews.llvm.org/D59119#1486753>, @efriedma wrote:

> If this is fixing a SPARC issue, please add an appropriate testcase for the SPARC backend.
>
> If iN ADDCARRY isn't legal, but iN is legal, it makes sense to handle that in SelectionDAGLegalize::ExpandNode, yes.  I thought we had that code already, but I guess not.  We should eventually implement ADDCARRY support for SPARC, but the code in SelectionDAGLegalize::ExpandNode will be useful for other targets which don't have an ADDCARRY-like instruction, like RISCV.


Made D61411 <https://reviews.llvm.org/D61411> to address this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59119