[cfe-commits] r163666 - /cfe/trunk/lib/Sema/SemaStmtAsm.cpp

[Chad Rosier]

Author: mcrosier
Date: Tue Sep 11 18:53:48 2012
New Revision: 163666

URL: http://llvm.org/viewvc/llvm-project?rev=163666&view=rev
Log:
[ms-inline asm] isDef/NumDefs in this context only refer to register definitions, not memory definitions.

Modified:
    cfe/trunk/lib/Sema/SemaStmtAsm.cpp

Modified: cfe/trunk/lib/Sema/SemaStmtAsm.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaStmtAsm.cpp?rev=163666&r1=163665&r2=163666&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaStmtAsm.cpp (original)
+++ cfe/trunk/lib/Sema/SemaStmtAsm.cpp Tue Sep 11 18:53:48 2012
@@ -595,10 +595,9 @@
       }
 
       const llvm::MCOperand &Op = Inst.getOperand(MCIdx);
-      bool isDef = NumDefs && (MCIdx < NumDefs);
 
       // Register/Clobber.
-      if (Op.isReg() && isDef) {
+      if (Op.isReg() && NumDefs && (MCIdx < NumDefs)) {
         std::string Reg;
         llvm::raw_string_ostream OS(Reg);
         IP->printRegName(OS, Op.getReg());
@@ -628,7 +627,7 @@
                                                   false, false);
             if (!Result.isInvalid()) {
               bool isMemDef = (i == 1) && Desc.mayStore();
-              if (isDef || isMemDef) {
+              if (isMemDef) {
                 Outputs.push_back(II);
                 OutputExprs.push_back(Result.take());
                 OutputExprNames.push_back(Name.str());