[PATCH] D124119: [InstCombine] Combine instructions of type or/and where AND masks can be combined.
    Sanjay Patel via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Jun  6 07:20:26 PDT 2022
    
    
  
spatel added a comment.
IIUC, the problem/fix (and it would be good to put something like this in the updated commit message for easier reference):
The previous revision/commit did not check one-use of an intermediate value that this transform re-uses. When that value has another use, an existing transform will try to invert the transform here. By adding one-use checks, we avoid the infinite loops seen with the earlier commit.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124119/new/
https://reviews.llvm.org/D124119
    
    
More information about the llvm-commits
mailing list