[PATCH] D146492: Add new printNumber() for size_t

[Paul Kirth via Phabricator via llvm-commits]

paulkirth added a comment.

Hi, sorry this is causing you a problem. I'm surprised our builders haven't picked this up. I'm also unable to reproduce this locally on a Mac or Linux machine.

Right now, it looks like CI also is running into trouble with this patch, so I'm not sure the additional overload is a good solution.

As a quick check, can you try casting  the value instead? Something along these lines:

  W.printNumber("Count", static_cast<uint64_t>(Count[I]));

If that works, we can probably change the container type away from `ArrayRef<size_t>` to use `uint64_t` or maybe even `uint32_t`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D146492/new/

https://reviews.llvm.org/D146492