Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline

Remove the unnecessary `is_nothrow_swappable` condition in `swap`.

Authored by mpark on Jul 6 2020, 7:10 PM.



Thanks to @lewissbaker who pointed out the unnecessary condition in Since this codepath does not
make use of swap anyway (that codepath is a different branch), we can safely
remove this condition and produce better codegen when all types are nothrow
movable but are potentially-throwing swappable.

See codegen in

Diff Detail

Event Timeline

mpark created this revision.Jul 6 2020, 7:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 6 2020, 7:10 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne accepted this revision.Jul 7 2020, 5:46 AM
This revision is now accepted and ready to land.Jul 7 2020, 5:46 AM
This revision was automatically updated to reflect the committed changes.