svn commit: r313171 - stable/11/contrib/llvm/lib/Transforms/Scalar

Dimitry Andric dim at FreeBSD.org
Fri Feb 3 19:15:55 UTC 2017


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.


More information about the svn-src-all mailing list