[PATCH] D43460: [X86] Add IMUL scheduling info on sandybridge, fix it on >=haswell.

[Craig Topper via Phabricator via llvm-commits]

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

LGTM. This makes sense to me. The extra uop on IMUL16rri/IMUL16rri8 is probably there to preserve the upper bits of the destination register since that operand is not part of the multiply operation. It makes sense that IMUL16rr wouldn't need that.


Repository:
  rL LLVM

https://reviews.llvm.org/D43460