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

[Robinson, Paul via llvm-commits]

Since we don't do DWARF64, and so should not have real 64-bit offsets, my bet is that it's endian-related.
--paulr

From: llvm-commits [mailto:llvm-commits-bounces at lists.llvm.org] On Behalf Of David Blaikie via llvm-commits
Sent: Monday, August 07, 2017 2:57 PM
To: Simon Dardis; llvm-commits at lists.llvm.org
Subject: Re: [llvm] r310262 - [DebugInfo][DWARF] Correct some usages of PRIx32 to PRIx64

Was it only failing on MIPS? Is it possible to test it more portably?

On Mon, Aug 7, 2017 at 11:51 AM Simon Dardis <Simon.Dardis at imgtec.com<mailto:Simon.Dardis at imgtec.com>> wrote:

From: David Blaikie [dblaikie at gmail.com<mailto:dblaikie at gmail.com>]
Sent: 07 August 2017 19:34
To: Simon Dardis; llvm-commits at lists.llvm.org<mailto:llvm-commits at lists.llvm.org>
Subject: Re: [llvm] r310262 - [DebugInfo][DWARF] Correct some usages of PRIx32 to PRIx64

> Test case?

This is already covered by:
 LLVM-Unit::DWARFDebugInfo.TestDwarfVerifyInvalidCURef
 LLVM-Unit::DWARFDebugInfo.TestDwarfVerifyInvalidStmtList

Which have been failing for some time on clang-cmake-mips(el):

http://lab.llvm.org:8011/builders/clang-cmake-mips/builds/4913

build 4914 when it's finished, should show those tests passing.

Thanks,
SImon

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170807/e94b3647/attachment.html>