[PATCH] D106607: [x86] improve CMOV codegen by hoisting add

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a comment.

Nice!

FTR - I'd started trying to get cmov-of-constants support into X86DAGToDAGISel::matchAddress (but maybe it should be in a LEA pass?) as I think we could then handle "add(x,add(y,cmov(20, 50))) -> add(add(x, 20), cmov(y,add(y,30)))" to help pull more of the math inside LEA?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D106607/new/

https://reviews.llvm.org/D106607