X-Git-Url: http://demsky.eecs.uci.edu/git/?a=blobdiff_plain;f=lib%2FVMCore%2FInlineAsm.cpp;h=524e294ab75f5b5a7efe3b1663413f0b4426246e;hb=2625f9b2e4388a957286063f6c7fe5406fd0ca7a;hp=d563e9557a6a234b3600293219dedc8d58afb358;hpb=96bb6226462536dc0fff4b5c0250613300abb9fb;p=oota-llvm.git diff --git a/lib/VMCore/InlineAsm.cpp b/lib/VMCore/InlineAsm.cpp index d563e9557a6..524e294ab75 100644 --- a/lib/VMCore/InlineAsm.cpp +++ b/lib/VMCore/InlineAsm.cpp @@ -57,7 +57,7 @@ bool InlineAsm::ConstraintInfo::Parse(const std::string &Str, // Initialize Type = isInput; isEarlyClobber = false; - hasMatchingInput = false; + MatchingInput = -1; isCommutative = false; isIndirect = false; @@ -127,8 +127,13 @@ bool InlineAsm::ConstraintInfo::Parse(const std::string &Str, Type != isInput) return true; // Invalid constraint number. + // If Operand N already has a matching input, reject this. An output + // can't be constrained to the same value as multiple inputs. + if (ConstraintsSoFar[N].hasMatchingInput()) + return true; + // Note that operand #n has a matching input. - ConstraintsSoFar[N].hasMatchingInput = true; + ConstraintsSoFar[N].MatchingInput = ConstraintsSoFar.size(); } else { // Single letter constraint. Codes.push_back(std::string(I, I+1)); @@ -183,15 +188,18 @@ bool InlineAsm::Verify(const FunctionType *Ty, const std::string &ConstStr) { if (Constraints.empty() && !ConstStr.empty()) return false; unsigned NumOutputs = 0, NumInputs = 0, NumClobbers = 0; + unsigned NumIndirect = 0; for (unsigned i = 0, e = Constraints.size(); i != e; ++i) { switch (Constraints[i].Type) { case InlineAsm::isOutput: + if ((NumInputs-NumIndirect) != 0 || NumClobbers != 0) + return false; // outputs before inputs and clobbers. if (!Constraints[i].isIndirect) { - if (NumInputs || NumClobbers) return false; // outputs come first. ++NumOutputs; break; } + ++NumIndirect; // FALLTHROUGH for Indirect Outputs. case InlineAsm::isInput: if (NumClobbers) return false; // inputs before clobbers.