[clang] [NFC] Replace bool <= bool comparison (PR #102948)
Mital Ashok via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 12 11:22:08 PDT 2024
https://github.com/MitalAshok created https://github.com/llvm/llvm-project/pull/102948
Closes #102912
>From fea4def3e66e7934718bab9d288094f7cbc5e4b7 Mon Sep 17 00:00:00 2001
From: Mital Ashok <mital at mitalashok.co.uk>
Date: Mon, 12 Aug 2024 19:03:53 +0100
Subject: [PATCH] [NFC] Replace bool <= bool
---
clang/lib/Sema/SemaOverload.cpp | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/clang/lib/Sema/SemaOverload.cpp b/clang/lib/Sema/SemaOverload.cpp
index 52f640eb96b73b..1ce0fa091938d7 100644
--- a/clang/lib/Sema/SemaOverload.cpp
+++ b/clang/lib/Sema/SemaOverload.cpp
@@ -509,7 +509,7 @@ NarrowingKind StandardConversionSequence::getNarrowingKind(
constexpr auto CanRepresentAll = [](bool FromSigned, unsigned FromWidth,
bool ToSigned, unsigned ToWidth) {
return (FromWidth < ToWidth + (FromSigned == ToSigned)) &&
- (FromSigned <= ToSigned);
+ !(FromSigned && !ToSigned);
};
if (CanRepresentAll(FromSigned, FromWidth, ToSigned, ToWidth))
@@ -542,7 +542,7 @@ NarrowingKind StandardConversionSequence::getNarrowingKind(
// If the bit-field width was dependent, it might end up being small
// enough to fit in the target type (unless the target type is unsigned
// and the source type is signed, in which case it will never fit)
- if (DependentBitField && (FromSigned <= ToSigned))
+ if (DependentBitField && !(FromSigned && !ToSigned))
return NK_Dependent_Narrowing;
// Otherwise, such a conversion is always narrowing
More information about the cfe-commits
mailing list