This is an archive of the discontinued LLVM Phabricator instance.

Demanded elements support for vector GEPs
ClosedPublic

Authored by reames on Jan 24 2019, 11:45 AM.

Details

Summary

GEPs can produce either scalar or vector results. If we're extracting only a subset of the vector lanes, simplifying the operands is helpful in eliminating redundant computation, and (eventually) allowing further optimizations.

This is the first concrete patch split out of https://reviews.llvm.org/D57140. If you're curious where this is going, that combined patch may be useful for context.

Diff Detail

Repository
rL LLVM

Event Timeline

reames created this revision.Jan 24 2019, 11:45 AM
spatel accepted this revision.Jan 26 2019, 8:01 AM

I haven't looked at vector geps very much, but the code change looks obviously ok, and the tests are thorough. LGTM.

This revision is now accepted and ready to land.Jan 26 2019, 8:01 AM
This revision was automatically updated to reflect the committed changes.