This is an archive of the discontinued LLVM Phabricator instance.

[Hexagon] Add support for nontemporal loads and stores on HVX
ClosedPublic

Authored by michaelwu on Jul 6 2017, 9:21 PM.

Details

Summary

This makes __builtin_nontemporal_load and __builtin_nontemporal_store work properly for HVX registers.

Diff Detail

Repository
rL LLVM

Event Timeline

michaelwu created this revision.Jul 6 2017, 9:21 PM
michaelwu edited the summary of this revision. (Show Details)
michaelwu retitled this revision from Add support for nontemporal loads and stores on HVX to [Hexagon] Add support for nontemporal loads and stores on HVX.Jul 7 2017, 8:25 AM
kparzysz added inline comments.Jul 10 2017, 1:33 PM
test/CodeGen/Hexagon/hvx-nontemporal.ll
1 ↗(On Diff #105580)
  1. Add -march=hexagon.
  2. Please use function attributes instead of -mv60 and -enable-hexagon-hvx-double.
kparzysz edited edge metadata.Jul 10 2017, 1:33 PM

With those changes, LGTM.

michaelwu updated this revision to Diff 105970.Jul 11 2017, 1:19 AM

Test adjusted per review comments.

michaelwu marked an inline comment as done.Jul 11 2017, 1:19 AM
kparzysz accepted this revision.Jul 11 2017, 5:48 AM

Perfect! Thanks.

This revision is now accepted and ready to land.Jul 11 2017, 5:48 AM

I don't have commit access. Can someone land this for me? Thanks!

This revision was automatically updated to reflect the committed changes.