[PATCH] D130466: [LICM] - Add option to allow data races

[Shubham Narlawar via Phabricator via llvm-commits]

gsocshubham updated this revision to Diff 452161.
gsocshubham added a comment.
Herald added a subscriber: ormris.

Sink store if architecture supports thread model single.

1. If thread model single is set, then store is sinked.
2. For those architecture which do not support -mthread-model single, they could make use of command line argument `-thread-model-single`

Let me know suggestions on whether to keep (2) approach or not.


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

https://reviews.llvm.org/D130466

Files:
  clang/lib/CodeGen/BackendUtil.cpp
  llvm/include/llvm/Passes/PassBuilder.h
  llvm/include/llvm/Transforms/Scalar/LICM.h
  llvm/include/llvm/Transforms/Utils/LoopUtils.h
  llvm/lib/Passes/PassBuilder.cpp
  llvm/lib/Passes/PassBuilderPipelines.cpp
  llvm/lib/Transforms/Scalar/LICM.cpp
  llvm/test/Transforms/LICM/promote-sink-store.ll
  llvm/test/Transforms/LICM/without-allow-data-race.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D130466.452161.patch
Type: text/x-patch
Size: 20289 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220812/0ba73c15/attachment.bin>