This is an archive of the discontinued LLVM Phabricator instance.

[AArch64][SVE] Asm: Support for contiguous PRF prefetch instructions.
ClosedPublic

Authored by sdesmalen on May 10 2018, 2:14 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

sdesmalen created this revision.May 10 2018, 2:14 AM

I might be good to put commit-message/description.

I might be good to put commit-message/description.

Hi Javed, for this patch I'm not sure what to put in the description that isn't already described by the title?

SjoerdMeijer accepted this revision.May 14 2018, 1:22 AM

Looks okay to me.

test/MC/AArch64/SVE/prfb-diagnostics.s
8 ↗(On Diff #146100)

Nit not related to this patch: but "pre-fetch" here and "prefetch" below is a bit inconsistent?

This revision is now accepted and ready to land.May 14 2018, 1:22 AM

Suggestion for commit message - "Add support and tests for contiguous Prefetch instructions different variants."
The problem i find sometimes with long title is that it it truncated.

This revision was automatically updated to reflect the committed changes.
sdesmalen marked an inline comment as done.May 16 2018, 12:55 AM