From 612f31284e423904d3afa255ed20ac36f2a1e010 Mon Sep 17 00:00:00 2001 From: David Majnemer Date: Wed, 10 Dec 2014 02:36:41 +0000 Subject: DataLayout: Provide nicer diagnostics for malformed strings llvm-svn: 223911 --- llvm/lib/IR/DataLayout.cpp | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) (limited to 'llvm/lib') diff --git a/llvm/lib/IR/DataLayout.cpp b/llvm/lib/IR/DataLayout.cpp index 4e0c066b1c8..cde393777a6 100644 --- a/llvm/lib/IR/DataLayout.cpp +++ b/llvm/lib/IR/DataLayout.cpp @@ -199,6 +199,8 @@ static std::pair split(StringRef Str, char Separator) { std::pair Split = Str.split(Separator); if (Split.second.empty() && Split.first != Str) report_fatal_error("Trailing separator in datalayout string"); + if (!Split.second.empty() && Split.first.empty()) + report_fatal_error("Expected token before separator in datalayout string"); return Split; } @@ -297,6 +299,9 @@ void DataLayout::parseSpecifier(StringRef Desc) { "Sized aggregate specification in datalayout string"); // ABI alignment. + if (Rest.empty()) + report_fatal_error( + "Missing alignment specification in datalayout string"); Split = split(Rest, ':'); unsigned ABIAlign = inBytes(getInt(Tok)); @@ -328,8 +333,12 @@ void DataLayout::parseSpecifier(StringRef Desc) { break; } case 'm': - assert(Tok.empty()); - assert(Rest.size() == 1); + if (!Tok.empty()) + report_fatal_error("Unexpected trailing characters after mangling specifier in datalayout string"); + if (Rest.empty()) + report_fatal_error("Expected mangling specifier in datalayout string"); + if (Rest.size() > 1) + report_fatal_error("Unknown mangling specifier in datalayout string"); switch(Rest[0]) { default: report_fatal_error("Unknown mangling in datalayout string"); -- cgit v1.2.3