[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 23:31:11 PDT 2023


momo5502 added a comment.

I thought about static_cast'ing `Val` instead of changing the format string. In case the underlying type of `Val` changes in the future, one would otherwise have to adapt the format string as well.
Or is there a way to emit a compiler warning if the format string mismatches? You mentioned marking `llvm::format` with a format attribute, @eli.friedman? Does that trigger a warning?


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