[PATCH] D41348: [ARM] Lower unsigned saturation to USAT

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 18 12:49:50 PST 2017


fhahn added reviewers: eastig, SjoerdMeijer, javed.absar.
fhahn added a comment.

Thanks, this looks good to me, with the test extension. A nice extension to the SSAT case! Adding a couple of additional Arm people, to have a quick look.



================
Comment at: test/CodeGen/ARM/usat.ll:1
+; RUN: llc -mtriple=armv4t-eabi %s -o - | FileCheck %s --check-prefix=CHECK --check-prefix=V4T
+; RUN: llc -mtriple=armv6t2-eabi %s -o - | FileCheck %s --check-prefix=CHECK --check-prefix=V6T2
----------------
This instruction should be available in ARMv6+ as well. Could you extend the test to cover that as well?


https://reviews.llvm.org/D41348





More information about the llvm-commits mailing list