Skip to content

Commit 8f25e43

Browse files
authored
[NFC] Rename hasSameElementsOrSplat to hasSameNumElementsOrSplat (#133183)
Makes it less confusing as this function only matches the number of elements
1 parent 13615f7 commit 8f25e43

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

mlir/lib/IR/BuiltinAttributes.cpp

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -589,7 +589,7 @@ static APInt readBits(const char *rawData, size_t bitPos, size_t bitWidth) {
589589
/// Returns true if 'values' corresponds to a splat, i.e. one element, or has
590590
/// the same element count as 'type'.
591591
template <typename Values>
592-
static bool hasSameElementsOrSplat(ShapedType type, const Values &values) {
592+
static bool hasSameNumElementsOrSplat(ShapedType type, const Values &values) {
593593
return (values.size() == 1) ||
594594
(type.getNumElements() == static_cast<int64_t>(values.size()));
595595
}
@@ -901,7 +901,7 @@ bool DenseElementsAttr::classof(Attribute attr) {
901901

902902
DenseElementsAttr DenseElementsAttr::get(ShapedType type,
903903
ArrayRef<Attribute> values) {
904-
assert(hasSameElementsOrSplat(type, values));
904+
assert(hasSameNumElementsOrSplat(type, values));
905905

906906
Type eltType = type.getElementType();
907907

@@ -985,7 +985,7 @@ DenseElementsAttr DenseElementsAttr::get(ShapedType type,
985985

986986
DenseElementsAttr DenseElementsAttr::get(ShapedType type,
987987
ArrayRef<bool> values) {
988-
assert(hasSameElementsOrSplat(type, values));
988+
assert(hasSameNumElementsOrSplat(type, values));
989989
assert(type.getElementType().isInteger(1));
990990

991991
std::vector<char> buff(llvm::divideCeil(values.size(), CHAR_BIT));
@@ -1020,15 +1020,15 @@ DenseElementsAttr DenseElementsAttr::get(ShapedType type,
10201020
DenseElementsAttr DenseElementsAttr::get(ShapedType type,
10211021
ArrayRef<APInt> values) {
10221022
assert(type.getElementType().isIntOrIndex());
1023-
assert(hasSameElementsOrSplat(type, values));
1023+
assert(hasSameNumElementsOrSplat(type, values));
10241024
size_t storageBitWidth = getDenseElementStorageWidth(type.getElementType());
10251025
return DenseIntOrFPElementsAttr::getRaw(type, storageBitWidth, values);
10261026
}
10271027
DenseElementsAttr DenseElementsAttr::get(ShapedType type,
10281028
ArrayRef<std::complex<APInt>> values) {
10291029
ComplexType complex = llvm::cast<ComplexType>(type.getElementType());
10301030
assert(llvm::isa<IntegerType>(complex.getElementType()));
1031-
assert(hasSameElementsOrSplat(type, values));
1031+
assert(hasSameNumElementsOrSplat(type, values));
10321032
size_t storageBitWidth = getDenseElementStorageWidth(complex) / 2;
10331033
ArrayRef<APInt> intVals(reinterpret_cast<const APInt *>(values.data()),
10341034
values.size() * 2);
@@ -1041,7 +1041,7 @@ DenseElementsAttr DenseElementsAttr::get(ShapedType type,
10411041
DenseElementsAttr DenseElementsAttr::get(ShapedType type,
10421042
ArrayRef<APFloat> values) {
10431043
assert(llvm::isa<FloatType>(type.getElementType()));
1044-
assert(hasSameElementsOrSplat(type, values));
1044+
assert(hasSameNumElementsOrSplat(type, values));
10451045
size_t storageBitWidth = getDenseElementStorageWidth(type.getElementType());
10461046
return DenseIntOrFPElementsAttr::getRaw(type, storageBitWidth, values);
10471047
}
@@ -1050,7 +1050,7 @@ DenseElementsAttr::get(ShapedType type,
10501050
ArrayRef<std::complex<APFloat>> values) {
10511051
ComplexType complex = llvm::cast<ComplexType>(type.getElementType());
10521052
assert(llvm::isa<FloatType>(complex.getElementType()));
1053-
assert(hasSameElementsOrSplat(type, values));
1053+
assert(hasSameNumElementsOrSplat(type, values));
10541054
ArrayRef<APFloat> apVals(reinterpret_cast<const APFloat *>(values.data()),
10551055
values.size() * 2);
10561056
size_t storageBitWidth = getDenseElementStorageWidth(complex) / 2;

0 commit comments

Comments
 (0)