[LLVM][tools] Remove unnecessary newline from error message

[sunshaoce]

The previous missing a newline:

$ llc --mattr=help
llc: error: unable to get target for 'unknown', see --version and --triple.$

[llvmbot]

@llvm/pr-subscribers-mc

Author: Shao-Ce SUN (sunshaoce)

Changes

This reverts commit 696eb3a.

The previous commit caused missing a newline:

$ llc --mattr=help
llc: error: unable to get target for 'unknown', see --version and --triple.$

---

Full diff: https://github.com/llvm/llvm-project/pull/120037.diff

1 Files Affected:

• (modified) llvm/lib/MC/TargetRegistry.cpp (+3-2)

diff --git a/llvm/lib/MC/TargetRegistry.cpp b/llvm/lib/MC/TargetRegistry.cpp
index 3be6f1d4634990..b1c1f93791f679 100644
--- a/llvm/lib/MC/TargetRegistry.cpp
+++ b/llvm/lib/MC/TargetRegistry.cpp
@@ -142,8 +142,9 @@ const Target *TargetRegistry::lookupTarget(StringRef ArchName,
     std::string TempError;
     TheTarget = TargetRegistry::lookupTarget(TheTriple.getTriple(), TempError);
     if (!TheTarget) {
-      Error = "unable to get target for '" + TheTriple.getTriple() +
-              "', see --version and --triple.";
+      Error = "unable to get target for '"
+            + TheTriple.getTriple()
+            + "', see --version and --triple.\n";
       return nullptr;
     }
   }

[github-actions]

✅ With the latest revision this PR passed the C/C++ code formatter.

[arichardson]

I don't think the error should contain a newline, probably better to add it to LLC. I don't think any of the other errors have it.

[sunshaoce]

Yes, you are right. I made modifications in llc.

[arichardson]

Do any other users of this function need to be updated?

[sunshaoce]

I tried searching for the lookupTarget with three arguments, and it seems there is no problem now.