[PATCH] D56534: [Verifier] Add verification of unaligned atomic load/store

James Y Knight via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 17 12:58:08 PST 2019


jyknight added a comment.

I believe that instead of this patch, the LangRef should be updated to say:

1. Misaligned atomics in the IR are allowed, and work correctly (but will most likely be inefficient).
2. Unless all accesses to a given byte of memory are done with the same address and size, there can be no guarantee of atomicity. (that is: no overlapping accesses, and no mixed-size accesses)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56534/new/

https://reviews.llvm.org/D56534





More information about the llvm-commits mailing list