This is an archive of the discontinued LLVM Phabricator instance.

Add fenv.h header
ClosedPublic

Authored by EricWF on Feb 4 2019, 6:09 PM.

Details

Summary

Some implementations of fenv.h use macros to define the functions they provide. This can cause problems when std::fegetround() is spelled in source.

This patch adds a fenv.h header to libc++ for the sole purpose of turning those macros into real functions.

Diff Detail

Repository
rCXX libc++

Event Timeline

EricWF created this revision.Feb 4 2019, 6:09 PM

Gentle ping.

rsmith accepted this revision.Feb 11 2019, 3:37 PM

LGTM

This revision is now accepted and ready to land.Feb 11 2019, 3:37 PM
This revision was automatically updated to reflect the committed changes.