[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