[PATCH] D130466: [LICM] - Add option to allow data races
Shubham Narlawar via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 26 01:53:13 PDT 2022
gsocshubham 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.
Can you please suggest a better name for the flag instead of `allow-data-races`?
Understood. I will move this flag to `TargetTransformInfo`
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130466/new/
https://reviews.llvm.org/D130466
More information about the llvm-commits
mailing list