Improve readability of error messages for VADD in ARM assembler.

Fixes readability of error messages, based on comment in CL: https://codereview.chromium.org/1652173002 BUG=None R=stichnot@chromium.org Review URL: https://codereview.chromium.org/1654803003 .

Improve readability of error messages for VADD in ARM assembler.
c0487128 · Karl Schimpf · 2c0764e2 · c0487128 · c0487128
Commit c0487128 authored Feb 01, 2016 by Karl Schimpf
Show whitespace changes
Inline Side-by-side

Showing with 4 additions and 5 deletions

IceAssemblerARM32.cpp src/IceAssemblerARM32.cpp +2 -3

IceInstARM32.cpp src/IceInstARM32.cpp +2 -2

No files found.
--- a/src/IceAssemblerARM32.cpp
+++ b/src/IceAssemblerARM32.cpp
@@ -157,9 +157,8 @@ IValueT encodeElmtType(Type ElmtTy) {
  case IceType_i64:
    return 3;
  default:
-    llvm::report_fatal_error(
+    llvm::report_fatal_error("SIMD op: Don't understand element type " +
-        std::string("SIMD op: Don't understand element type ") +
+                             std::string(typeString(ElmtTy)));
-        typeString(ElmtTy));
  }
 }

--- a/src/IceInstARM32.cpp
+++ b/src/IceInstARM32.cpp
@@ -621,8 +621,8 @@ template <> void InstARM32Vadd::emitIAS(const Cfg *Func) const {
  case IceType_v8i1:
  case IceType_v16i1:
  case IceType_NUM:
-    llvm::report_fatal_error(std::string("Vadd not defined on type ") +
+    llvm::report_fatal_error("Vadd not defined on type " +
-                             typeString(DestTy));
+                             std::string(typeString(DestTy)));
    break;
  case IceType_v16i8:
  case IceType_v8i16: