svn commit: r312993 - head/contrib/llvm/lib/Transforms/Scalar
Dimitry Andric
dim at FreeBSD.org
Mon Jan 30 18:35:26 UTC 2017
Author: dim
Date: Mon Jan 30 18:35:24 2017
New Revision: 312993
URL: https://svnweb.freebsd.org/changeset/base/312993
Log:
Pull in r279454 from upstream llvm trunk (by James Molloy):
[SROA] Remove incorrect assertion
Confirmed with aprantl, this assertion is incorrect - code can get
here (for example 80-bit FP types) and if it does it's benign. This
is exposed by a completely unrelated patch of mine, so stop the
compiler falling over.
Original differential: http://reviews.llvm.org/D16187
aprantl's advice to remove assertion:
http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20160815/382129.html
This should fix assertions when building the math/opensolaris-libm port.
Reported by: marino
MFC after: 3 days
Modified:
head/contrib/llvm/lib/Transforms/Scalar/SROA.cpp
Modified: head/contrib/llvm/lib/Transforms/Scalar/SROA.cpp
==============================================================================
--- head/contrib/llvm/lib/Transforms/Scalar/SROA.cpp Mon Jan 30 16:32:53 2017 (r312992)
+++ head/contrib/llvm/lib/Transforms/Scalar/SROA.cpp Mon Jan 30 18:35:24 2017 (r312993)
@@ -4040,9 +4040,6 @@ bool SROA::splitAlloca(AllocaInst &AI, A
Size = std::min(Size, AbsEnd - Start);
}
PieceExpr = DIB.createBitPieceExpression(Start, Size);
- } else {
- assert(Pieces.size() == 1 &&
- "partition is as large as original alloca");
}
// Remove any existing dbg.declare intrinsic describing the same alloca.
More information about the svn-src-head
mailing list