This is an archive of the discontinued LLVM Phabricator instance.

[ARM][NEON] Combine base address updates for vld1x intrinsics
ClosedPublic

Authored by krisb on May 20 2021, 10:18 AM.

Diff Detail

Event Timeline

krisb created this revision.May 20 2021, 10:18 AM
krisb requested review of this revision.May 20 2021, 10:18 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 20 2021, 10:18 AM
krisb updated this revision to Diff 346792.May 20 2021, 10:31 AM

Remove llvm/test/CodeGen/ARM/pr45824.ll as it is no longer needed.

dmgreen accepted this revision.May 24 2021, 3:54 AM

Looks great. I was wondering on D102256 if you were going to do loads too, but didn't want to presume.

Thanks for the patch. LGTM.

This revision is now accepted and ready to land.May 24 2021, 3:54 AM

@dmgreen thank you for reviewing this!
I think I can also take a look at vldxdup intrinsics by the end of the week. They seem to be the last group of intrinsics that CombineBaseUpdate() hasn't been supporting yet.

krisb updated this revision to Diff 347598.May 25 2021, 12:54 AM

Applied a few clang-format suggestions. NFC.

This revision was landed with ongoing or failed builds.May 25 2021, 2:07 AM
This revision was automatically updated to reflect the committed changes.
llvm/test/CodeGen/ARM/arm-vld1.ll