[PATCH] D153575: [TargetLowering] Better code generation for ISD::SADDSAT/SSUBSAT when operand sign is known
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 22 12:01:38 PDT 2023
nikic added reviewers: dmgreen, RKSimon.
nikic added a comment.
Looks reasonable to me.
================
Comment at: llvm/test/CodeGen/AArch64/aarch64-saturating-arithmetic.ll:2
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py UTC_ARGS: --version 2
+; RUN: llc -verify-machineinstrs -mtriple=aarch64-linux-gnu -O3 -o - %s | FileCheck %s --check-prefixes=CHECK
+
----------------
You can drop `-verify-machineinstrs`, `-O3` and `--check-prefixes=CHECK` here. `-verify-machineinstrs` is enabled under EXPENSIVE_CHECKS, and doesn't need to be added to most tests. `-O3` rarely differs from the default of `-O2` for llc. `--check-prefixes=CHECK` is already the default.
================
Comment at: llvm/test/CodeGen/AArch64/aarch64-saturating-arithmetic.ll:209
+declare i64 @llvm.smax(i64, i64)
+declare i64 @llvm.smin(i64, i64)
----------------
Can you please commit the tests with baseline checks and then rebase the patch over that? See https://llvm.org/docs/TestingGuide.html#precommit-workflow-for-tests.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153575/new/
https://reviews.llvm.org/D153575
More information about the llvm-commits
mailing list