[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