+  /// \brief True if the assembly statement does not have any input or output
+  /// operands.
   bool IsSimple;

Why not name it "HasNoInputOutputOperands" then?

--Sean Silva

On Mon, Aug 27, 2012 at 5:14 PM, Chad Rosier <[email protected]> wrote:
> Author: mcrosier
> Date: Mon Aug 27 16:14:01 2012
> New Revision: 162699
>
> URL: http://llvm.org/viewvc/llvm-project?rev=162699&view=rev
> Log:
> [ms-inline asm] Add a few comments.
>
> Modified:
>     cfe/trunk/include/clang/AST/Stmt.h
>
> Modified: cfe/trunk/include/clang/AST/Stmt.h
> URL: 
> http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/Stmt.h?rev=162699&r1=162698&r2=162699&view=diff
> ==============================================================================
> --- cfe/trunk/include/clang/AST/Stmt.h (original)
> +++ cfe/trunk/include/clang/AST/Stmt.h Mon Aug 27 16:14:01 2012
> @@ -1368,7 +1368,12 @@
>  class AsmStmt : public Stmt {
>  protected:
>    SourceLocation AsmLoc;
> +  /// \brief True if the assembly statement does not have any input or output
> +  /// operands.
>    bool IsSimple;
> +
> +  /// \brief If true, treat this inline assembly as having side effects.
> +  /// This assembly statement should not be optimized, deleted or moved.
>    bool IsVolatile;
>
>    unsigned NumOutputs;
>
>
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to