Page MenuHomePhabricator
Feed Advanced Search

Jun 23 2018

ronniemitchell3232 added a comment to D47921: [ARM] Allow CMPZ transforms even if the input has multiple uses..

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.

Jun 23 2018, 3:36 AM

Jun 16 2018

ronniemitchell3232 added a comment to D47921: [ARM] Allow CMPZ transforms even if the input has multiple uses..

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.

Jun 16 2018, 3:32 PM