This is an archive of the discontinued LLVM Phabricator instance.

[GlobalISel][AArch64] Use getOpcodeDef instead of findMIFromReg
ClosedPublic

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

Details

Summary

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

Repository
rL LLVM

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.