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

[Chad Rosier]

Author: mcrosier
Date: Wed Sep 12 19:20:37 2012
New Revision: 163758

URL: http://llvm.org/viewvc/llvm-project?rev=163758&view=rev
Log:
Add a few FIXMEs.

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=163758&r1=163757&r2=163758&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaStmtAsm.cpp (original)
+++ cfe/trunk/lib/Sema/SemaStmtAsm.cpp Wed Sep 12 19:20:37 2012
@@ -568,6 +568,8 @@
       if (Operands[i]->isToken() || Operands[i]->isImm())
         continue;
 
+      // FIXME: The getMCInstOperandNum() function does not work with tied 
+      // operands or custom converters.
       unsigned NumMCOperands;
       unsigned MCIdx = TargetParser->getMCInstOperandNum(Kind, Inst, Operands,
                                                          i, NumMCOperands);
@@ -614,6 +616,7 @@
               ExprResult Result = ActOnIdExpression(getCurScope(), SS, Loc, Id,
                                                     false, false);
               if (!Result.isInvalid()) {
+                // FIXME: Determine the proper constraints.
                 bool isMemDef = (i == 1) && Desc.mayStore();
                 if (isMemDef) {
                   Outputs.push_back(II);