[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