[PATCH] D46073: [DagCombine] De Morgan laws: 'nand' logic with an inverted operand

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 2 10:24:21 PDT 2018


spatel added a comment.

In https://reviews.llvm.org/D46073#1080839, @lebedev.ri wrote:

> - Rebased ontop of revised tests.
> - Handle `nor` pattern too.
> - Still not sure how to handle `vandnpd`/`vandnps` breakage.


I lost track of where we are in the masked merge odyssey. :)
I think this is the next step, but we have to avoid regressing the x86 andnps tests? 
Can we use TLI.hasAndNot() to predicate the transform? Ie, if you have 'andn', then there's no point trying to eliminate a 'not' that will get merged with another instruction?


Repository:
  rL LLVM

https://reviews.llvm.org/D46073





More information about the llvm-commits mailing list