[all-commits] [llvm/llvm-project] b84696: Fix the type of offset that broke 32-bit flang-rt ...

[Daniel Chen via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b84696db745e127cc6fb1d49bbf39f3c0819b6d9
      https://github.com/llvm/llvm-project/commit/b84696db745e127cc6fb1d49bbf39f3c0819b6d9
  Author: Daniel Chen <cdchen at ca.ibm.com>
  Date:   2025-07-08 (Tue, 08 Jul 2025)

  Changed paths:
    M flang-rt/lib/runtime/assign.cpp
    M flang-rt/lib/runtime/type-info.cpp

  Log Message:
  -----------
  Fix the type of offset that broke 32-bit flang-rt build to use `uint64_t` consistently (#147359)

The recent change of `flang-rt` has code like `std::size_t
offset{offset_};`.
It broke the 32-bit `flang-rt` build because `Component::offset_` is of
type `uint64_t` but `size_t` varies.
Clang complains
```
error: non-constant-expression cannot be narrowed from type 'std::uint64_t' (aka 'unsigned long long') to 'std::size_t' (aka 'unsigned long') in initializer list [-Wc++11-narrowing]
  143 |   std::size_t offset{offset_};
      |                      ^~~~~~~

```

This patch is to use the consistent `uint64_t` for offset.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications