This is an archive of the discontinued LLVM Phabricator instance.

[AArch64][SVE] Bail out of performPostLD1Combine for scalable types
ClosedPublic

Authored by kmclaughlin on Jun 26 2020, 11:08 AM.

Details

Summary

performPostLD1Combine will introduce either a LD1LANEpost
or LD1DUPpost node, which will cause selection failure if the
return type is a scalable vector.

Diff Detail

Event Timeline

kmclaughlin created this revision.Jun 26 2020, 11:08 AM
efriedma accepted this revision.Jun 26 2020, 3:05 PM

I was looking into this too; looks like you beat me to it. :)

LGTM with a minor comment on the test.

llvm/test/CodeGen/AArch64/sve-ld-post-inc.ll
8

Please add CHECK lines for the actual load/arithmetic you expect to be generated, even if they aren't really "important" to the crash.

This revision is now accepted and ready to land.Jun 26 2020, 3:05 PM
This revision was automatically updated to reflect the committed changes.