[PATCH] D54424: [DWARF] Do not use PRIx32 for printing uint64_t values

Simon Atanasyan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 12 12:05:44 PST 2018


atanasyan added a comment.

In https://reviews.llvm.org/D54424#1295873, @dblaikie wrote:

> In https://reviews.llvm.org/D54424#1295850, @atanasyan wrote:
>
> > In https://reviews.llvm.org/D54424#1295814, @dblaikie wrote:
> >
> > > Any chance of a test case?
> >
> >
> > I hope the following test case starts to pass on mips buildbot:
> >  http://lab.llvm.org:8011/builders/llvm-mips-linux/builds/3739/steps/test-llvm/logs/FAIL%3A%20LLVM%3A%3Adebug_addr.ll
>
>
> Could this also be tested cross-platform (on any platform that's building the mips backend)? (ie: a version of this test that 'REQUIRES: ' the mips target and specifically targets it)


I cannot figure out how to do that. In fact we test various libc. The following code (compiled by gcc and clang) works fine on x86_64 in both 32/64-bit executables but shows incorrect output on mips 32-bit.

uint64_t V = 1;
printf("0x%8.8" PRIx32 "\n", V);


Repository:
  rL LLVM

https://reviews.llvm.org/D54424





More information about the llvm-commits mailing list