[PATCH] D95325: [AArch64][GlobalISel] Make G_SADDE and G_SSUBE legal

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 26 10:01:41 PST 2021


paquette accepted this revision.
paquette added a comment.
This revision is now accepted and ready to land.

LGTM with test nit



================
Comment at: llvm/test/CodeGen/AArch64/GlobalISel/legalize-sadde.mir:21
+    %3:_(s1) = G_TRUNC %2
+    %4:_(s32), %5:_(s1) = G_SADDE %0, %1, %3
+    %6:_(s32) = G_ANYEXT %3
----------------
Nit: named vregs are easier to read


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95325



More information about the llvm-commits mailing list