Date: Fri, 3 Feb 2017 19:15:53 +0000 (UTC) From: Dimitry Andric <dim@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-11@freebsd.org Subject: svn commit: r313171 - stable/11/contrib/llvm/lib/Transforms/Scalar Message-ID: <201702031915.v13JFrWO039880@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: dim Date: Fri Feb 3 19:15:53 2017 New Revision: 313171 URL: https://svnweb.freebsd.org/changeset/base/313171 Log: MFC r312993: 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 Modified: stable/11/contrib/llvm/lib/Transforms/Scalar/SROA.cpp Directory Properties: stable/11/ (props changed) Modified: stable/11/contrib/llvm/lib/Transforms/Scalar/SROA.cpp ============================================================================== --- stable/11/contrib/llvm/lib/Transforms/Scalar/SROA.cpp Fri Feb 3 19:12:54 2017 (r313170) +++ stable/11/contrib/llvm/lib/Transforms/Scalar/SROA.cpp Fri Feb 3 19:15:53 2017 (r313171) @@ -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.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201702031915.v13JFrWO039880>