[PATCH] D95032: [AArch64][GlobalISel] Make G_USUBO legal and select it

[Cassie Jones via Phabricator via llvm-commits]

porglezomp updated this revision to Diff 318685.
porglezomp added a comment.

Fix instruction selection for G_USUBO

G_USUBO needs to be checked for overflow with the LO condition code instead of
the HS condition code. This should hopefully resolve the UBSan issue.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95032

Files:
  llvm/lib/Target/AArch64/GISel/AArch64InstructionSelector.cpp
  llvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp
  llvm/test/CodeGen/AArch64/GlobalISel/legalize-add.mir
  llvm/test/CodeGen/AArch64/GlobalISel/legalize-sub.mir
  llvm/test/CodeGen/AArch64/GlobalISel/legalizer-info-validation.mir
  llvm/test/CodeGen/AArch64/GlobalISel/select-saddo.mir
  llvm/test/CodeGen/AArch64/GlobalISel/select-ssubo.mir
  llvm/test/CodeGen/AArch64/GlobalISel/select-usubo.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D95032.318685.patch
Type: text/x-patch
Size: 25566 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210123/c025f9b5/attachment.bin>