[PATCH] D63004: [TargetLowering] Simplify (ctpop x) == 1
    Sanjay Patel via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Sun Jun  9 08:41:40 PDT 2019
    
    
  
spatel added a comment.
In D63004#1535595 <https://reviews.llvm.org/D63004#1535595>, @xbolva00 wrote:
> @spatel Thanks for review
>
> What about current revision? Is it OK for you?
Seems ok, but you didn't answer the request to add tests for another target. We want to have more confidence that this is universally good, so something besides x86 should be tested.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63004/new/
https://reviews.llvm.org/D63004
    
    
More information about the llvm-commits
mailing list