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

Junhee Yoo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 21 20:18:43 PDT 2023

junhee-yoo added a comment.

In D146492#4211754 <https://reviews.llvm.org/D146492#4211754>, @paulkirth wrote:

> @junhee-yoo you may want to try your code w/ top-of-tree again. looks like someone added some `static_cast` to those APIs earlier today, so you may no longer have an issue. I didn't even notice that had landed earlier when I proposed changing to `uint32_t`.

Thanks for sharing. 👍 https://reviews.llvm.org/rG064e2497e2ebe9ac30ac96923a26a52484300fdf it looks same as your first suggestion and it has been finished successfully with the config that you told me.

Meanwhile, it seems your Mac CPU (x86_64) and my one(M1 <https://reviews.llvm.org/M1>: aarch64) are different so it makes different results.

> Homebrew clang version 14.0.6
> Target: arm64-apple-darwin22.3.0
> Thread model: posix
> InstalledDir: /opt/homebrew/opt/llvm at 14/bin

I guess this ambiguity problem only happens in M1 <https://reviews.llvm.org/M1>(aarch64). I've tried to build a parent of https://reviews.llvm.org/rG064e2497e2ebe9ac30ac96923a26a52484300fdf with your config to confirm my assumption and ambiguous problem occurred.

  rG LLVM Github Monorepo



More information about the llvm-commits mailing list