Page MenuHomePhabricator

[NEON] Support VLD1xN intrinsics in AArch32 mode (LLVM part)
ClosedPublic

Authored by kosarev on May 20 2018, 11:28 AM.

Details

Summary

We currently support them only in AArch64. The NEON Reference, however, says they are 'ARMv7, ARMv8' intrinsics.

Diff Detail

Repository
rL LLVM

Event Timeline

kosarev created this revision.May 20 2018, 11:28 AM
kosarev retitled this revision from [NEON] Support VLD1xN intrinsics in AArch32 mode to [NEON] Support VLD1xN intrinsics in AArch32 mode (LLVM part).May 20 2018, 11:29 AM
SjoerdMeijer accepted this revision.May 31 2018, 6:46 AM

Like I said in D47121, I agree that these intrinsics are available in v7/A32/A64.
This change looks reasonable to me.

lib/Target/ARM/ARMExpandPseudoInsts.cpp
468 ↗(On Diff #147717)

typo: intructions

This revision is now accepted and ready to land.May 31 2018, 6:46 AM
This revision was automatically updated to reflect the committed changes.