[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