[llvm] [KnownBits] Make `avg{Ceil,Floor}S` optimal (PR #110329)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 30 11:46:39 PDT 2024
================
@@ -306,14 +306,14 @@ TEST(KnownBitsTest, BinaryExhaustive) {
return KnownBits::add(Known1, Known2);
},
[](const APInt &N1, const APInt &N2) { return N1 + N2; },
- /*CheckOptimality=*/false);
+ /*CheckOptimality=*/true);
testBinaryOpExhaustive(
"sub",
[](const KnownBits &Known1, const KnownBits &Known2) {
return KnownBits::sub(Known1, Known2);
},
[](const APInt &N1, const APInt &N2) { return N1 - N2; },
- /*CheckOptimality=*/false);
+ /*CheckOptimality=*/true);
----------------
goldsteinn wrote:
No, this was just motivated by the fact that I noticed we had incorrectly labelled a lot of the tests. I pushed an NFC change to correctly label them.
https://github.com/llvm/llvm-project/pull/110329
More information about the llvm-commits
mailing list