[PATCH] D46493: [DagCombiner] Not all 'andn''s work with immediates.

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 7 10:24:42 PDT 2018


spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

LGTM - but I want to point out that the mca analysis may not be accurate for these sequences yet. We don't have the zero idiom (xor %eax, %eax) recognition yet ( https://bugs.llvm.org/show_bug.cgi?id=36671 ), so the stats for those cases are probably pessimistic.


Repository:
  rL LLVM

https://reviews.llvm.org/D46493





More information about the llvm-commits mailing list