This is an archive of the discontinued LLVM Phabricator instance.

[x86] fix 256-bit vector store splitting to honor 'volatile'
ClosedPublic

Authored by spatel on May 27 2019, 5:15 PM.

Details

Summary

Forking this out of the discussion in D62498 (and optimistically assuming that will be approved, so adding the helper function here).
The LangRef says:
"the backend should never split or merge target-legal volatile load/store instructions."

Diff Detail

Event Timeline

spatel created this revision.May 27 2019, 5:15 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 27 2019, 5:15 PM
This revision is now accepted and ready to land.May 27 2019, 7:25 PM
lebedev.ri accepted this revision.May 28 2019, 4:14 AM

(you can just base on diff ontop of another one, the base needn't be the current git master / svn trunk)

(you can just base on diff ontop of another one, the base needn't be the current git master / svn trunk)

Still learning how to git...
I was going to rebase D62498 on top of this, but ran out of time yesterday. Now that this is approved, I'll just push this and update there.

This revision was automatically updated to reflect the committed changes.