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

Thorsten via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 25 12:40:25 PDT 2022


tschuett added a comment.

In D130466#3677171 <https://reviews.llvm.org/D130466#3677171>, @efriedma wrote:

> I don't really want to introduce an option named "allow-data-races".
>
> If we want to do something based on the assumption the program is single-threaded, the option should be named that.  And ideally, it shouldn't be specific to LICM; it should be something any pass can query from TargetTransformInfo.

Or could it be in LangRef instead. Correct multi-threaded code would have used locks or atomics.


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

https://reviews.llvm.org/D130466



More information about the llvm-commits mailing list