[PATCH] D144614: [LegalizeTypes][RISCV] Add a special case to ExpandIntRes_UADDSUBO for (uaddo X, 1).

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 23 09:58:51 PST 2023


reames added a comment.

Both this and the related patch for ADD seem like they could be covered by a single set of DAG combine rules.  Specifically:
a + 1 < a ==> a + 1 == 0
a - 1 > a ==> a + 1 == 0

(With the comparisons in the notation above being setcc nodes.)

Would doing the more general form and removing the expansion specific bits be worthwhile?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144614



More information about the llvm-commits mailing list