[PATCH] D67399: [ARM] Follow AACPS standard for volatile bitfields
John McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Sep 12 17:12:11 PDT 2019
rjmccall added a comment.
The exact sequence of volatile accesses is observable behavior, and it's the ABI's right to define correct behavior for compliant implementations, so we do need to do this.
Diogo, IRGen breaks up bit-field storage units in CGRecordLowering::accumulateBitFields. I'm sure there's some reasonable way to add target-specific adjustments to that so that we break things up by storage unit.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67399/new/
https://reviews.llvm.org/D67399
More information about the cfe-commits
mailing list