[PATCH] D58967: [DAGCombiner] Add SADDO/SSUBO combine support

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

I think we'd do better by making a helper for the common code rather than duplicating. These things tend to diverge because someone will come along later and add a fold for 1 opcode but forget to add the corresponding fold for its sibling. This might've already happened because we allow vectors for the signed ops, but not the unsigned ops...or that's the planned follow-up?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D58967