diff --git a/llvm/lib/Remarks/BitstreamRemarkParser.cpp b/llvm/lib/Remarks/BitstreamRemarkParser.cpp index 99a82e1ee3af4..1cfde11922295 100644 --- a/llvm/lib/Remarks/BitstreamRemarkParser.cpp +++ b/llvm/lib/Remarks/BitstreamRemarkParser.cpp @@ -278,19 +278,20 @@ Expected BitstreamParserHelper::isRemarkBlock() { return isBlock(Stream, META_BLOCK_ID); } -static Error validateMagicNumber(StringRef Magic) { - if (Magic != remarks::ContainerMagic) +static Error validateMagicNumber(StringRef MagicNumber) { + if (MagicNumber != remarks::ContainerMagic) return createStringError(std::make_error_code(std::errc::invalid_argument), "Unknown magic number: expecting %s, got %.4s.", - remarks::ContainerMagic.data(), Magic.data()); + remarks::ContainerMagic.data(), MagicNumber.data()); return Error::success(); } static Error advanceToMetaBlock(BitstreamParserHelper &Helper) { - Expected> Magic = Helper.parseMagic(); - if (!Magic) - return Magic.takeError(); - if (Error E = validateMagicNumber(StringRef(Magic->data(), Magic->size()))) + Expected> MagicNumber = Helper.parseMagic(); + if (!MagicNumber) + return MagicNumber.takeError(); + if (Error E = validateMagicNumber( + StringRef(MagicNumber->data(), MagicNumber->size()))) return E; if (Error E = Helper.parseBlockInfoBlock()) return E; @@ -309,11 +310,12 @@ remarks::createBitstreamParserFromMeta( StringRef Buf, Optional StrTab, Optional ExternalFilePrependPath) { BitstreamParserHelper Helper(Buf); - Expected> Magic = Helper.parseMagic(); - if (!Magic) - return Magic.takeError(); + Expected> MagicNumber = Helper.parseMagic(); + if (!MagicNumber) + return MagicNumber.takeError(); - if (Error E = validateMagicNumber(StringRef(Magic->data(), Magic->size()))) + if (Error E = validateMagicNumber( + StringRef(MagicNumber->data(), MagicNumber->size()))) return std::move(E); auto Parser = diff --git a/llvm/lib/Remarks/RemarkFormat.cpp b/llvm/lib/Remarks/RemarkFormat.cpp index c7fce151eff0f..5006421a3c638 100644 --- a/llvm/lib/Remarks/RemarkFormat.cpp +++ b/llvm/lib/Remarks/RemarkFormat.cpp @@ -32,9 +32,9 @@ Expected llvm::remarks::parseFormat(StringRef FormatStr) { return Result; } -Expected llvm::remarks::magicToFormat(StringRef Magic) { +Expected llvm::remarks::magicToFormat(StringRef MagicStr) { auto Result = - StringSwitch(Magic) + StringSwitch(MagicStr) .StartsWith("--- ", Format::YAML) // This is only an assumption. .StartsWith(remarks::Magic, Format::YAMLStrTab) .StartsWith(remarks::ContainerMagic, Format::Bitstream) @@ -42,6 +42,6 @@ Expected llvm::remarks::magicToFormat(StringRef Magic) { if (Result == Format::Unknown) return createStringError(std::make_error_code(std::errc::invalid_argument), - "Unknown remark magic: '%s'", Magic.data()); + "Unknown remark magic: '%s'", MagicStr.data()); return Result; }