This is an archive of the discontinued LLVM Phabricator instance.

Do not optimize atomic load to non-atomic memcmp
ClosedPublic

Authored by christylee on Sep 12 2018, 10:59 AM.

Diff Detail

Event Timeline

christylee created this revision.Sep 12 2018, 10:59 AM
trentxintong accepted this revision.Sep 12 2018, 11:02 AM

LGTM. @efriedma, does this look good to you ? I think we have to have similar treatments for memcpyopt in the past.

This revision is now accepted and ready to land.Sep 12 2018, 11:02 AM

Thanks for the review! @efriedma

And of course thank you to @trentxintong :)

courbet accepted this revision.Sep 12 2018, 11:10 PM

Thanks.

This revision was automatically updated to reflect the committed changes.