[PATCH] D152790: [ARM] Fix codegen of unaligned volatile load/store of i64
Dave Green via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 13 02:53:13 PDT 2023
dmgreen added a reviewer: vhscampos.
dmgreen added a comment.
Hello
Should the condition be bases on Subtarget->allowsUnalignedMem too? I'm not sure exactly in which architectures LDRD required an alignment of 8 and when it required an alignment of 4. I believe in most cases it is 4, with some early architectures requiring 8 if Subtarget->allowsUnalignedMem was false.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152790/new/
https://reviews.llvm.org/D152790
More information about the llvm-commits
mailing list