[libcxx-commits] [PATCH] D135781: [libc++] Assume that builtins for math.h functions are available

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a comment.

In D135781#3892483 <https://reviews.llvm.org/D135781#3892483>, @EricWF wrote:

> Obviously, this would be less complex and nicer if we could assume their presence, but I have no doubt this more complicated code was written because we couldn't at some point.

Our compiler requirements have changed a lot since 2012, and nowadays the compilers we support are tested in the CI. I assume @philnik's reasoning was that "I'll try doing it, and if the CI is happy, then it means that all supported compilers are happy". That's what we normally do and it has led to large simplifications (e.g. in how we implement `type_traits`). I agree it would make sense to record that rationale (or whatever the rationale is in case I'm wrong) in the commit message though.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135781/new/

https://reviews.llvm.org/D135781