This is an archive of the discontinued LLVM Phabricator instance.

Fix some problems with sized deletion in libc++
AbandonedPublic

Authored by mclow.lists on Feb 17 2015, 7:46 AM.

Details

Reviewers
lvoufo
rsmith
Summary

The sized deletion routines were recently added to libc++ (r229281). There are some problems with the current implementation. This fixes some of those problems, but there are others extant:

  1. How we're going to support vendors who ship new dylibs infrequently; compatibility with existing dylibs. This is especially important if clang is going to implicitly generate calls to these routines.
  1. The original commit had no tests.

Diff Detail

Event Timeline

mclow.lists retitled this revision from to Fix some problems with sized deletion in libc++.
mclow.lists updated this object.
mclow.lists edited the test plan for this revision. (Show Details)
mclow.lists added reviewers: lvoufo, rsmith.
mclow.lists added a subscriber: Unknown Object (MLST).
mclow.lists added inline comments.
include/new
147

This change (and the one below) are two simple changes that the diff obscures:

  1. Move the two variants that have a size_t param together.
  2. Wrap them in an ifdef.
mclow.lists abandoned this revision.Jun 23 2015, 8:03 AM

This stuff has been superseded by LWG issue #2458