[libcxx-commits] [PATCH] D60234: Added std::assoc_legendre and std::legendre [sf.cmath]

[Andre Brand via Phabricator via libcxx-commits]

thebrandre added a comment.

No worries. :-)
I know that this is time-consuming and I am well aware that is not a high-priority feature. And now I had time to reconsider and find my own mistakes. ;-)
The integration should be now much cleaner than my first draft.

Btw .... I should also point out that this draft implicitly checks the flag _LIBCPP_NO_EXCEPTIONS (via __throw_domain_error) in the src folder. I keep wondering which of these #defines may actually be used in cpp files. Should this be avoided so there's no reason to build different versions of the library?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60234