[llvm] r289545 - Use more detailed assertion messages in the code introduced by r289538
Artur Pilipenko via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 13 08:26:15 PST 2016
Author: apilipenko
Date: Tue Dec 13 10:26:15 2016
New Revision: 289545
URL: http://llvm.org/viewvc/llvm-project?rev=289545&view=rev
Log:
Use more detailed assertion messages in the code introduced by r289538
Modified:
llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
Modified: llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp?rev=289545&r1=289544&r2=289545&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp (original)
+++ llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp Tue Dec 13 10:26:15 2016
@@ -4446,7 +4446,8 @@ collectByteProviders(SDValue Op, bool Ch
Result.insert(Result.begin(), ByteShift, ByteProvider::getZero());
Result.insert(Result.end(), Original->begin(),
std::prev(Original->end(), ByteShift));
- assert(Result.size() == ByteWidth && "sanity");
+ assert(Result.size() == ByteWidth &&
+ "Computed width doesn't match from type width");
return Result;
}
case ISD::ZERO_EXTEND: {
@@ -4460,7 +4461,8 @@ collectByteProviders(SDValue Op, bool Ch
Result.insert(Result.begin(), Original->begin(), Original->end());
Result.insert(Result.end(), ByteWidth - NarrowByteWidth,
ByteProvider::getZero());
- assert(Result.size() == ByteWidth && "sanity");
+ assert(Result.size() == ByteWidth &&
+ "Computed width doesn't match from type width");
return Result;
}
case ISD::LOAD: {
@@ -4469,7 +4471,8 @@ collectByteProviders(SDValue Op, bool Ch
L->getExtensionType() != ISD::NON_EXTLOAD)
return None;
- assert(BitWidth == L->getMemoryVT().getSizeInBits() && "sanity");
+ assert(BitWidth == L->getMemoryVT().getSizeInBits() &&
+ "For non-extend loads widths must be the same");
SmallVector<ByteProvider, 4> Result(ByteWidth);
for (unsigned i = 0; i < ByteWidth; i++)
@@ -4517,7 +4520,8 @@ SDValue DAGCombiner::MatchLoadCombine(SD
return SDValue();
auto &Bytes = Res.getValue();
unsigned ByteWidth = Bytes.size();
- assert(VT.getSizeInBits() == ByteWidth * 8 && "sanity");
+ assert(VT.getSizeInBits() == ByteWidth * 8 &&
+ "collectByteProviders computed width differs from type width");
auto LittleEndianByteAt = [](unsigned BW, unsigned i) { return i; };
auto BigEndianByteAt = [](unsigned BW, unsigned i) { return BW - i - 1; };
More information about the llvm-commits
mailing list