diff options
| author | Chris Lattner <sabre@nondot.org> | 2010-04-23 16:29:58 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2010-04-23 16:29:58 +0000 |
| commit | 1a8f394a1fe76c93f8793802cc3fd025560a1123 (patch) | |
| tree | 3c94a88784bd2c1dce7ef1b667fb8db752ad4e15 /clang/lib/AST/Stmt.cpp | |
| parent | 83eb5032c09db7b6d8e0909a2c2a5a36e9bbcff8 (diff) | |
| download | bcm5719-llvm-1a8f394a1fe76c93f8793802cc3fd025560a1123.tar.gz bcm5719-llvm-1a8f394a1fe76c93f8793802cc3fd025560a1123.zip | |
david conrad points out that {|} in inline assembly on arm are not asm
variants. This fixes neon inline asm which my patch for PR6780 broke.
llvm-svn: 102181
Diffstat (limited to 'clang/lib/AST/Stmt.cpp')
| -rw-r--r-- | clang/lib/AST/Stmt.cpp | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/clang/lib/AST/Stmt.cpp b/clang/lib/AST/Stmt.cpp index 97023820e24..ee4a74a6125 100644 --- a/clang/lib/AST/Stmt.cpp +++ b/clang/lib/AST/Stmt.cpp @@ -19,6 +19,7 @@ #include "clang/AST/Type.h" #include "clang/AST/ASTContext.h" #include "clang/AST/ASTDiagnostic.h" +#include "clang/Basic/TargetInfo.h" #include <cstdio> using namespace clang; @@ -240,6 +241,8 @@ unsigned AsmStmt::AnalyzeAsmString(llvm::SmallVectorImpl<AsmStringPiece>&Pieces, // asm string. std::string CurStringPiece; + bool HasVariants = !C.Target.hasNoAsmVariants(); + while (1) { // Done with the string? if (CurPtr == StrEnd) { @@ -251,9 +254,9 @@ unsigned AsmStmt::AnalyzeAsmString(llvm::SmallVectorImpl<AsmStringPiece>&Pieces, char CurChar = *CurPtr++; switch (CurChar) { case '$': CurStringPiece += "$$"; continue; - case '{': CurStringPiece += "$("; continue; - case '|': CurStringPiece += "$|"; continue; - case '}': CurStringPiece += "$)"; continue; + case '{': CurStringPiece += (HasVariants ? "$(" : "{"); continue; + case '|': CurStringPiece += (HasVariants ? "$|" : "|"); continue; + case '}': CurStringPiece += (HasVariants ? "$)" : "}"); continue; case '%': break; default: |
