[PATCH] D153800: [ARM] Adjust strd/ldrd codegen alignment requirements

Maurice Heumann via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 11 12:20:14 PDT 2023


momo5502 added a comment.

In D153800#4490748 <https://reviews.llvm.org/D153800#4490748>, @efriedma wrote:

> `snprintf(buf, bufsize, "%d", 1LL)` has undefined behavior.

Jup. that's it.

All these formats should be something along the lines of `%llx` not `%x`: https://github.com/llvm/llvm-project/blob/main/llvm/lib/DebugInfo/DWARF/DWARFTypePrinter.cpp#L427
Changing that fixes the issue.

Thanks for your help :D I will check if there are more occurances and prepare a proper fix tomorrow.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153800



More information about the llvm-commits mailing list