[PATCH] D22806: [ARM] Improve error messages for .arch_extension directive

[Oliver Stannard via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL276748: [ARM] Improve error messages for .arch_extension directive (authored by olista01).

Changed prior to commit:
  https://reviews.llvm.org/D22806?vs=65505&id=65516#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D22806

Files:
  llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
  llvm/trunk/test/MC/ARM/directive-arch_extension-unsupported.s

Index: llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
===================================================================
--- llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
+++ llvm/trunk/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
@@ -10505,7 +10505,7 @@
   MCAsmParser &Parser = getParser();
 
   if (getLexer().isNot(AsmToken::Identifier)) {
-    Error(getLexer().getLoc(), "unexpected token");
+    Error(getLexer().getLoc(), "expected architecture extension name");
     Parser.eatToEndOfStatement();
     return false;
   }
@@ -10520,15 +10520,19 @@
     Name = Name.substr(2);
   }
   unsigned FeatureKind = ARM::parseArchExt(Name);
-  if (FeatureKind == ARM::AEK_INVALID)
+  if (FeatureKind == ARM::AEK_INVALID) {
     Error(ExtLoc, "unknown architectural extension: " + Name);
+    return false;
+  }
 
   for (const auto &Extension : Extensions) {
     if (Extension.Kind != FeatureKind)
       continue;
 
-    if (Extension.Features.none())
-      report_fatal_error("unsupported architectural extension: " + Name);
+    if (Extension.Features.none()) {
+      Error(ExtLoc, "unsupported architectural extension: " + Name);
+      return false;
+    }
 
     if ((getAvailableFeatures() & Extension.ArchCheck) != Extension.ArchCheck) {
       Error(ExtLoc, "architectural extension '" + Name + "' is not "
Index: llvm/trunk/test/MC/ARM/directive-arch_extension-unsupported.s
===================================================================
--- llvm/trunk/test/MC/ARM/directive-arch_extension-unsupported.s
+++ llvm/trunk/test/MC/ARM/directive-arch_extension-unsupported.s
@@ -0,0 +1,25 @@
+@ RUN: not llvm-mc -triple armv7--none-eabi -filetype asm -o /dev/null 2>&1 %s | FileCheck %s
+
+	.arch_extension os
+CHECK: error: unsupported architectural extension: os
+
+	.arch_extension iwmmxt
+CHECK: error: unsupported architectural extension: iwmmxt
+
+	.arch_extension iwmmxt2
+CHECK: error: unsupported architectural extension: iwmmxt2
+
+	.arch_extension maverick
+CHECK: error: unsupported architectural extension: maverick
+
+	.arch_extension xscale
+CHECK: error: unsupported architectural extension: xscale
+
+	.arch_extension invalid_extension_name
+CHECK: error: unknown architectural extension: invalid_extension_name
+
+	.arch_extension 42
+CHECK: error: expected architecture extension name
+
+	.arch_extension
+CHECK: error: expected architecture extension name


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22806.65516.patch
Type: text/x-patch
Size: 2398 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160726/55084997/attachment.bin>