diff options
| author | Rafael Espindola <rafael.espindola@gmail.com> | 2011-01-01 21:47:03 +0000 |
|---|---|---|
| committer | Rafael Espindola <rafael.espindola@gmail.com> | 2011-01-01 21:47:03 +0000 |
| commit | 478abcab3daef06b71303063da323ffc649f56ea (patch) | |
| tree | e2226d36d442a2bf9817e6b86f487b800b113195 /clang/lib/Sema/SemaDecl.cpp | |
| parent | 5734edc109d2cd76626f2f3746c394e0b304ae1f (diff) | |
| download | bcm5719-llvm-478abcab3daef06b71303063da323ffc649f56ea.tar.gz bcm5719-llvm-478abcab3daef06b71303063da323ffc649f56ea.zip | |
Produce a better error message for invalid register names.
llvm-svn: 122670
Diffstat (limited to 'clang/lib/Sema/SemaDecl.cpp')
| -rw-r--r-- | clang/lib/Sema/SemaDecl.cpp | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp index 62262bc60b5..abd63198d47 100644 --- a/clang/lib/Sema/SemaDecl.cpp +++ b/clang/lib/Sema/SemaDecl.cpp @@ -2995,8 +2995,12 @@ Sema::ActOnVariableDeclarator(Scope *S, Declarator &D, DeclContext *DC, if (Expr *E = (Expr*)D.getAsmLabel()) { // The parser guarantees this is a string. StringLiteral *SE = cast<StringLiteral>(E); + llvm::StringRef Label = SE->getString(); + if (S->getFnParent() != 0 && + !Context.Target.isValidGCCRegisterName(Label)) + Diag(E->getExprLoc(), diag::err_asm_unknown_register_name) << Label; NewVD->addAttr(::new (Context) AsmLabelAttr(SE->getStrTokenLoc(0), - Context, SE->getString())); + Context, Label)); } // Diagnose shadowed variables before filtering for scope. |
