[llvm-commits] [llvm] r81272 - in /llvm/trunk/lib: ExecutionEngine/JIT/TargetSelect.cpp Support/TargetRegistry.cpp

[Daniel Dunbar]

Author: ddunbar
Date: Tue Sep  8 18:32:35 2009
New Revision: 81272

URL: http://llvm.org/viewvc/llvm-project?rev=81272&view=rev
Log:
Improve JIT error message for users crazy enough to use -march with JIT, and
mention -version in messages about missing targets.

Modified:
    llvm/trunk/lib/ExecutionEngine/JIT/TargetSelect.cpp
    llvm/trunk/lib/Support/TargetRegistry.cpp

Modified: llvm/trunk/lib/ExecutionEngine/JIT/TargetSelect.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/JIT/TargetSelect.cpp?rev=81272&r1=81271&r2=81272&view=diff

==============================================================================
--- llvm/trunk/lib/ExecutionEngine/JIT/TargetSelect.cpp (original)
+++ llvm/trunk/lib/ExecutionEngine/JIT/TargetSelect.cpp Tue Sep  8 18:32:35 2009
@@ -26,7 +26,8 @@
 using namespace llvm;
 
 static cl::opt<std::string>
-MArch("march", cl::desc("Architecture to generate assembly for (see --version)"));
+MArch("march",
+      cl::desc("Architecture to generate assembly for (see --version)"));
 
 static cl::opt<std::string>
 MCPU("mcpu",
@@ -61,7 +62,8 @@
     }
 
     if (!TheTarget) {
-      errs() << "JIT: error: invalid target '" << MArch << "'.\n";
+      *ErrorStr = "No available targets are compatible with this -march, "
+        "see -version for the available targets.\n";
       return 0;
     }
 

Modified: llvm/trunk/lib/Support/TargetRegistry.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/TargetRegistry.cpp?rev=81272&r1=81271&r2=81272&view=diff

==============================================================================
--- llvm/trunk/lib/Support/TargetRegistry.cpp (original)
+++ llvm/trunk/lib/Support/TargetRegistry.cpp Tue Sep  8 18:32:35 2009
@@ -40,7 +40,8 @@
   }
 
   if (!Best) {
-    Error = "No available targets are compatible with this triple";
+    Error = "No available targets are compatible with this triple, "
+      "see -version for the available targets.";
     return 0;
   }