[llvm] r310262 - [DebugInfo][DWARF] Correct some usages of PRIx32 to PRIx64

[Hans Wennborg via llvm-commits]

Merged to 5.0 in r311196.

On Mon, Aug 7, 2017 at 8:37 AM, Simon Dardis via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> Author: sdardis
> Date: Mon Aug  7 08:37:57 2017
> New Revision: 310262
>
> URL: http://llvm.org/viewvc/llvm-project?rev=310262&view=rev
> Log:
> [DebugInfo][DWARF] Correct some usages of PRIx32 to PRIx64
>
> These lead to tests failing spuriously as the values after being rendered to a
> string were incorrect.
>
> Reviewers: clayborg
>
> Differential Revision: https://reviews.llvm.org/D36319
>
> Modified:
>     llvm/trunk/lib/DebugInfo/DWARF/DWARFVerifier.cpp
>
> Modified: llvm/trunk/lib/DebugInfo/DWARF/DWARFVerifier.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/DebugInfo/DWARF/DWARFVerifier.cpp?rev=310262&r1=310261&r2=310262&view=diff
> ==============================================================================
> --- llvm/trunk/lib/DebugInfo/DWARF/DWARFVerifier.cpp (original)
> +++ llvm/trunk/lib/DebugInfo/DWARF/DWARFVerifier.cpp Mon Aug  7 08:37:57 2017
> @@ -253,7 +253,7 @@ unsigned DWARFVerifier::verifyDebugInfoA
>          ++NumErrors;
>          OS << "error: DW_AT_stmt_list offset is beyond .debug_line "
>                "bounds: "
> -           << format("0x%08" PRIx32, *SectionOffset) << "\n";
> +           << format("0x%08" PRIx64, *SectionOffset) << "\n";
>          Die.dump(OS, 0);
>          OS << "\n";
>        }
> @@ -292,7 +292,7 @@ unsigned DWARFVerifier::verifyDebugInfoF
>        if (CUOffset >= CUSize) {
>          ++NumErrors;
>          OS << "error: " << FormEncodingString(Form) << " CU offset "
> -           << format("0x%08" PRIx32, CUOffset)
> +           << format("0x%08" PRIx64, CUOffset)
>             << " is invalid (must be less than CU size of "
>             << format("0x%08" PRIx32, CUSize) << "):\n";
>          Die.dump(OS, 0);
> @@ -424,7 +424,7 @@ void DWARFVerifier::verifyDebugLineRows(
>        if (Row.Address < PrevAddress) {
>          ++NumDebugLineErrors;
>          OS << "error: .debug_line["
> -           << format("0x%08" PRIx32,
> +           << format("0x%08" PRIx64,
>                       *toSectionOffset(Die.find(DW_AT_stmt_list)))
>             << "] row[" << RowIndex
>             << "] decreases in address from previous row:\n";
> @@ -439,7 +439,7 @@ void DWARFVerifier::verifyDebugLineRows(
>        if (Row.File > MaxFileIndex) {
>          ++NumDebugLineErrors;
>          OS << "error: .debug_line["
> -           << format("0x%08" PRIx32,
> +           << format("0x%08" PRIx64,
>                       *toSectionOffset(Die.find(DW_AT_stmt_list)))
>             << "][" << RowIndex << "] has invalid file index " << Row.File
>             << " (valid values are [1," << MaxFileIndex << "]):\n";
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits