[PATCH] D47921: [ARM] Allow CMPZ transforms even if the input has multiple uses.

Ronnie Mitchell via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 23 03:36:30 PDT 2018


ronniemitchell3232 added a comment.

In https://reviews.llvm.org/D47921#1126108, @SjoerdMeijer wrote:

> The initial commit showed problem in testing, and my fix added the check that the AND has a single use. Looks like indeed that we just forgot to remove the other check as I also don't see why it would be necessary. Thanks for fixing.





Repository:
  rL LLVM

https://reviews.llvm.org/D47921





More information about the llvm-commits mailing list