[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