Page MenuHomePhabricator

[GlobalISel][AArch64] Use getOpcodeDef instead of findMIFromReg

Authored by paquette on Jul 10 2019, 11:13 AM.



Some minor cleanup. Trying to make use of functions that live in Utils more.

getOpcodeDef in Utils does the same thing as findMIFromReg. It also looks through copies, which findMIFromReg didn't.

Delete findMIFromReg and use getOpcodeDef instead. This only happens in tryOptVectorDup right now.

Update opt-shuffle-splat to show that we can look through the copies now, too.

This doesn't really impact code size anywhere in real life as far as I've seen, but it's nice to improve code reuse.

Diff Detail


Event Timeline

paquette created this revision.Jul 10 2019, 11:13 AM
This revision is now accepted and ready to land.Jul 10 2019, 11:32 AM
This revision was automatically updated to reflect the committed changes.