This is an archive of the discontinued LLVM Phabricator instance.

[ARM MVE] Remove accidental 64-bit vst2/vld2 intrinsics.
ClosedPublic

Authored by simon_tatham on Nov 4 2019, 1:49 AM.

Details

Summary

ACLE defines no such intrinsic as vst2q_u64, and the MVE instruction
set has no corresponding instruction. But I had accidentally added
them to the fledgling <arm_mve.h> anyway, and if you used them, you'd
get a compiler crash.

Diff Detail

Event Timeline

simon_tatham created this revision.Nov 4 2019, 1:49 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 4 2019, 1:49 AM
dmgreen accepted this revision.Nov 4 2019, 4:09 AM

LGTM.

I was trying to think of a way to test this, but couldn't come up with anything sensible.

This revision is now accepted and ready to land.Nov 4 2019, 4:09 AM
This revision was automatically updated to reflect the committed changes.