[cfe-commits] r162699 - /cfe/trunk/include/clang/AST/Stmt.h

[Sean Silva]

+  /// \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 <mcrosier at apple.com> 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
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits