summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorOliver Stannard <oliver.stannard@arm.com>2017-10-24 09:46:56 +0000
committerOliver Stannard <oliver.stannard@arm.com>2017-10-24 09:46:56 +0000
commitce256a3a01b9f207f9a5eba6b313c008fa373241 (patch)
tree1f16201ae5f68b64b94763fafad55524b171161c
parent3356acacf7d102b66e814adf2a4e7094f3e4b581 (diff)
downloadbcm5719-llvm-ce256a3a01b9f207f9a5eba6b313c008fa373241.tar.gz
bcm5719-llvm-ce256a3a01b9f207f9a5eba6b313c008fa373241.zip
[ARM] Replace development diagnostics with normal DEBUG macro
* Remove the -arm-asm-parser-dev-diags option. * Use normal DEBUG(dbgs()) printing for the extra development information about missing diagnostics. Differential Revision: https://reviews.llvm.org/D39194 llvm-svn: 316423
-rw-r--r--llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp23
1 files changed, 9 insertions, 14 deletions
diff --git a/llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp b/llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
index 1e6c175e41d..0f31fd218e0 100644
--- a/llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
+++ b/llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
@@ -64,6 +64,8 @@
#include <utility>
#include <vector>
+#define DEBUG_TYPE "asm-parser"
+
using namespace llvm;
namespace {
@@ -85,11 +87,6 @@ static cl::opt<ImplicitItModeTy> ImplicitItMode(
static cl::opt<bool> AddBuildAttributes("arm-add-build-attributes",
cl::init(false));
-cl::opt<bool>
-DevDiags("arm-asm-parser-dev-diags", cl::init(false),
- cl::desc("Use extended diagnostics, which include implementation "
- "details useful for development"));
-
enum VectorLaneTy { NoLanes, AllLanes, IndexedLane };
class UnwindContext {
@@ -10200,18 +10197,16 @@ ARMAsmParser::FilterNearMisses(SmallVectorImpl<NearMissInfo> &NearMissesIn,
NearMissMessage Message;
Message.Loc = OperandLoc;
- raw_svector_ostream OS(Message.Message);
if (OperandDiag) {
- OS << OperandDiag;
+ Message.Message = OperandDiag;
} else if (I.getOperandClass() == InvalidMatchClass) {
- OS << "too many operands for instruction";
+ Message.Message = "too many operands for instruction";
} else {
- OS << "invalid operand for instruction";
- if (DevDiags) {
- OS << " class" << I.getOperandClass() << ", error "
- << I.getOperandError() << ", opcode "
- << MII.getName(I.getOpcode());
- }
+ Message.Message = "invalid operand for instruction";
+ DEBUG(dbgs() << "Missing diagnostic string for operand class " <<
+ getMatchClassName((MatchClassKind)I.getOperandClass())
+ << I.getOperandClass() << ", error " << I.getOperandError()
+ << ", opcode " << MII.getName(I.getOpcode()) << "\n");
}
NearMissesOut.emplace_back(Message);
break;
OpenPOWER on IntegriCloud