[llvm] c6a257f - [AArch64] Fix || Add brackets for || inside of assert
David Green via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 21 01:26:56 PDT 2024
Author: David Green
Date: 2024-06-21T09:26:45+01:00
New Revision: c6a257fe0fc9ba677d96929a34dedc5f243aad7d
URL: https://github.com/llvm/llvm-project/commit/c6a257fe0fc9ba677d96929a34dedc5f243aad7d
DIFF: https://github.com/llvm/llvm-project/commit/c6a257fe0fc9ba677d96929a34dedc5f243aad7d.diff
LOG: [AArch64] Fix || Add brackets for || inside of assert
Fixes:
llvm/lib/Target/AArch64/AArch64GenRegisterBankInfo.def:185:12: warning: suggest parentheses around ‘&&’ within ‘||’ [-Wparentheses]
assert(!Size.isScalable() || MinSize >= 128
~~~~~~~~~~~~~~
&& "Scalable vector types should have size of at least 128 bits");
Added:
Modified:
llvm/lib/Target/AArch64/AArch64GenRegisterBankInfo.def
Removed:
################################################################################
diff --git a/llvm/lib/Target/AArch64/AArch64GenRegisterBankInfo.def b/llvm/lib/Target/AArch64/AArch64GenRegisterBankInfo.def
index d243ca51cdcb9..82066b48c84b4 100644
--- a/llvm/lib/Target/AArch64/AArch64GenRegisterBankInfo.def
+++ b/llvm/lib/Target/AArch64/AArch64GenRegisterBankInfo.def
@@ -181,8 +181,8 @@ unsigned AArch64GenRegisterBankInfo::getRegBankBaseIdxOffset(unsigned RBIdx,
}
if (RBIdx == PMI_FirstFPR) {
const unsigned MinSize = Size.getKnownMinValue();
- assert(!Size.isScalable() || MinSize >= 128
- && "Scalable vector types should have size of at least 128 bits");
+ assert((!Size.isScalable() || MinSize >= 128) &&
+ "Scalable vector types should have size of at least 128 bits");
if (MinSize <= 16)
return 0;
if (MinSize <= 32)
More information about the llvm-commits
mailing list