[PATCH] D123991: [LangRef] Clarify load/store of non-byte-sized types
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 19 14:43:17 PDT 2022
nikic planned changes to this revision.
nikic added a comment.
In D123991#3460290 <https://reviews.llvm.org/D123991#3460290>, @craig.topper wrote:
> For loads, the default lowering in the backends assume that the padding bits are 0. Which is only possible if the store explicitly wrote those padding bits to 0. The default lowering of stores does that if it was stored with the same type. I know we've had bugs in X86 related to this. I'll see if I can find any.
Oh wow, you're right: https://llvm.godbolt.org/z/nf3898b89 These semantics aren't going to work then.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D123991/new/
https://reviews.llvm.org/D123991
More information about the llvm-commits
mailing list