[PATCH] D151878: [GlobalIsel][X86] Update legalization of G_ADD/G_SUB

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a comment.

The G_ADD legalizer tests are already in place for the legacy legalizer, I can add equivalent G_SUB coverage pretty easily - support for further opcodes needs better tests before they can be added. I'll start with the logic ops and integer multiplies.

If we wanted to add X86 specific legalizers, legalFor64Bit/legalForSSE2/AVX2 style wrappers etc. would probably be quite clean - can/should LegalizeRuleSet be overloaded on a per-target basis?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151878