[PATCH] D130466: [LICM] - Add option to force thread model single

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

In D130466#3844483 <https://reviews.llvm.org/D130466#3844483>, @arsenm wrote:

> How is this different from using syncscope("singlethread")?

They don't have any relation, syncscope is a property of atomic operations.

To remove all the LICM-specific aspects, the question here is basically whether it is safe to introduce a spurious (non-atomic) store of the form `store (load p), p` without introducing a data race.


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

https://reviews.llvm.org/D130466