From: Lang Hames <lhames@gmail.com>
Date: Thu, 4 Oct 2012 03:23:25 +0000 (+0000)
Subject: Fail early with a clear assert if an operation with multiple uses somehow ends
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=ff4ae6d4031962a7c4fdd344f40a91f0fcd26a64;p=clang

Fail early with a clear assert if an operation with multiple uses somehow ends
up being contracted during codegen.


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@165197 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/CodeGen/CGExprScalar.cpp b/lib/CodeGen/CGExprScalar.cpp
index 0d68ff5a69..0e8afb04d0 100644
--- a/lib/CodeGen/CGExprScalar.cpp
+++ b/lib/CodeGen/CGExprScalar.cpp
@@ -2045,11 +2045,15 @@ static Value* tryEmitFMulAdd(const BinOpInfo &op,
   // We have a potentially fusable op. Look for a mul on one of the operands.
   if (llvm::BinaryOperator* LHSBinOp = dyn_cast<llvm::BinaryOperator>(op.LHS)) {
     if (LHSBinOp->getOpcode() == llvm::Instruction::FMul) {
+      assert(LHSBinOp->getNumUses() == 0 &&
+             "Operations with multiple uses shouldn't be contracted.");
       return buildFMulAdd(LHSBinOp, op.RHS, CGF, Builder, false, isSub);
     }
   } else if (llvm::BinaryOperator* RHSBinOp =
                dyn_cast<llvm::BinaryOperator>(op.RHS)) {
     if (RHSBinOp->getOpcode() == llvm::Instruction::FMul) {
+      assert(RHSBinOp->getNumUses() == 0 &&
+             "Operations with multiple uses shouldn't be contracted.");
       return buildFMulAdd(RHSBinOp, op.LHS, CGF, Builder, isSub, false);
     }
   }