[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.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D146492/new/
https://reviews.llvm.org/D146492
More information about the llvm-commits
mailing list