[PATCH] D63806: [PowerPC][Peephole] Combine extsw and sldi after instruction selection

[Kai Luo via Phabricator via llvm-commits]

lkail marked an inline comment as done.
lkail added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCMIPeephole.cpp:1404
+  ++NumEXTSWAndSLDICombined;
+  ToErase = &MI;
+  return true;
----------------
hfinkel wrote:
> lkail wrote:
> > hfinkel wrote:
> > > Should we also erase SrcMI?
> > Yes. We can erase `SrcMI` here, also we can leave it to DCE pass. I have no idea which one is preferable.
> Might as well do it here.
Done.


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

https://reviews.llvm.org/D63806