Page MenuHomePhabricator

[MemCpyOpt] Don't hoist store that's not guaranteed to execute
ClosedPublic

Authored by nikic on Fri, Oct 9, 12:11 PM.

Details

Summary

MemCpyOpt can hoist stores while load+store pairs into memcpy. This hoisting can currently result in stores being executed that weren't guaranteed to execute in the original problem.

Diff Detail

Event Timeline

nikic created this revision.Fri, Oct 9, 12:11 PM
Herald added a project: Restricted Project. · View Herald TranscriptFri, Oct 9, 12:11 PM
nikic requested review of this revision.Fri, Oct 9, 12:11 PM
This revision is now accepted and ready to land.Fri, Oct 9, 12:25 PM