This is an archive of the discontinued LLVM Phabricator instance.

[libunwind] Use placement new to avoid dependency C++ library
ClosedPublic

Authored by phosek on Jan 25 2019, 12:08 PM.

Details

Summary

The rest of libunwind already uses placement new, these are the only
places where non-placement new is being used introducing undesirable
C++ library dependency.

Diff Detail

Repository
rL LLVM

Event Timeline

phosek created this revision.Jan 25 2019, 12:08 PM
ldionne added inline comments.Jan 25 2019, 1:13 PM
libunwind/src/libunwind.cpp
17 ↗(On Diff #183594)

Can we get rid of this header then?

18 ↗(On Diff #183594)

I think this header can go away regardless of the changes in this patch.

phosek updated this revision to Diff 183611.Jan 25 2019, 1:28 PM
phosek marked 2 inline comments as done.
ldionne accepted this revision.Jan 25 2019, 1:35 PM
This revision is now accepted and ready to land.Jan 25 2019, 1:35 PM
This revision was automatically updated to reflect the committed changes.