diff options
author | Jim Grosbach <grosbach@apple.com> | 2012-05-01 18:38:27 +0000 |
---|---|---|
committer | Jim Grosbach <grosbach@apple.com> | 2012-05-01 18:38:27 +0000 |
commit | 758e0cc94af7e146c948aadaf81e2667287c777b (patch) | |
tree | 98ea4eb080f96fb9afc1d7fd94365abe42d848ca /llvm/lib/MC/MCParser/AsmParser.cpp | |
parent | a0c53f147a9dd1d14edfedea758bb80d286bbccf (diff) | |
download | bcm5719-llvm-758e0cc94af7e146c948aadaf81e2667287c777b.tar.gz bcm5719-llvm-758e0cc94af7e146c948aadaf81e2667287c777b.zip |
MC: Unknown assembler directives are now hard errors.
Previously, an unsupported/unknown assembler directive issued a warning.
That's generally unsafe, and inconsistent with the behaviour of pretty
much every system assembler. Now that the MC assemblers are mature
enough to be the default on multiple targets, it's reasonable to
issue errors for these.
For target or platform directives that need to stay warnings, we
should add explicit handlers for them in, e.g., ELFAsmParser.cpp,
DarwinAsmParser.cpp, et. al., and issue the warning there.
rdar://9246275
llvm-svn: 155926
Diffstat (limited to 'llvm/lib/MC/MCParser/AsmParser.cpp')
-rw-r--r-- | llvm/lib/MC/MCParser/AsmParser.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/MC/MCParser/AsmParser.cpp b/llvm/lib/MC/MCParser/AsmParser.cpp index ec137e28913..25419916c71 100644 --- a/llvm/lib/MC/MCParser/AsmParser.cpp +++ b/llvm/lib/MC/MCParser/AsmParser.cpp @@ -1238,7 +1238,7 @@ bool AsmParser::ParseStatement() { if (!getTargetParser().ParseDirective(ID)) return false; - return Warning(IDLoc, "ignoring directive for now"); + return Error(IDLoc, "unknown directive"); } CheckForValidSection(); |