This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Remove math_win32.h
ClosedPublic

Authored by smeenai on Sep 23 2016, 12:49 PM.

Details

Summary

Visual Studio 2013 and onward have all the required functions in their
CRT headers, and we don't support older versions anymore.

Diff Detail

Repository
rL LLVM

Event Timeline

smeenai updated this revision to Diff 72330.Sep 23 2016, 12:49 PM
smeenai retitled this revision from to [libc++] Remove math_win32.h.
smeenai updated this object.
smeenai added reviewers: compnerd, EricWF, mclow.lists.
smeenai added a subscriber: cfe-commits.
EricWF accepted this revision.Sep 23 2016, 10:29 PM
EricWF edited edge metadata.

I'll take your word for it. Thanks for the patch.

This revision is now accepted and ready to land.Sep 23 2016, 10:29 PM

It's worth noting that the entirety of the deleted file was guarded by a #if ((_VC_CRT_MAJOR_VERSION-0) < 12) anyway (i.e. it would only be included on Visual Studio 2012 and below), so the code was already dead :)

This revision was automatically updated to reflect the committed changes.
libcxx/trunk/include/math.h