[PATCH] D153165: Select `G_SADDE`, `G_UADDE`, `G_SSUBE` and `G_USUBE` for AArch64
niwin anto via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Jun 17 01:01:16 PDT 2023
niwinanto added a comment.
In D153165#4429338 <https://reviews.llvm.org/D153165#4429338>, @tobias-stadler wrote:
> I fixed these same issues here https://reviews.llvm.org/D153164 2 minutes earlier. I do generate better code though, because I generate less instructions for carry-in setting and optimize these instructions away whenever possible. For >O0 I generate identical code to SelectionDag. I also have higher test coverage.
Ohh cool. Insane coincidence. My plan was to support missing instructions, other wise it can cause internal compiler crashes. I did not touch the optimization, hence less optimized code is expected.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153165/new/
https://reviews.llvm.org/D153165
More information about the llvm-commits
mailing list