[PATCH] D96676: [Test][AArch64] Test SADDE/SSUBE/UADDE/USUBE narrowing legalization

Cassie Jones via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 16 23:54:58 PST 2021


porglezomp added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/GlobalISel/legalize-usube.mir:38
+---
+name:            narrow_scalar_usube_big_nonpow2
+body:             |
----------------
arsenm wrote:
> What about a non-pow2 that also isn't divisible by the register size?
That isn't supported for even plain add/sub yet, so I think that should be a separate change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96676



More information about the llvm-commits mailing list