Page MenuHomePhabricator

Add fenv.h header
ClosedPublic

Authored by EricWF on Mon, Feb 4, 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.Mon, Feb 4, 6:09 PM

Gentle ping.

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

LGTM

This revision is now accepted and ready to land.Mon, Feb 11, 3:37 PM
Closed by commit rCXX353767: Add fenv.h header (authored by EricWF, committed by ). · Explain WhyMon, Feb 11, 3:46 PM
This revision was automatically updated to reflect the committed changes.