[PATCH] D34578: cmath: Support clang's -fdelayed-template-parsing

Hal Finkel via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 6 15:36:46 PDT 2017


hfinkel accepted this revision.
hfinkel added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D34578#801357, @dexonsmith wrote:

> Ping!  Hal, you committed r283051.  Do you have a problem with this?


It looks like you're just renaming `__libcpp_isnan` and friends to __libcpp_isnan_or_builtin` and similar. LGTM

> (Incidentally, I noticed that r283051 was optimizing the implementation of <complex>. I wonder why we have so much code in that header, instead of calling out to libc's already-optimized <complex.h> implementation?)

This is definitely worth looking at (especially given that we don't support complex of user-defined types with those functions). Would making complex<double> use _Complex double, and so on, be an API break? Would that matter?


https://reviews.llvm.org/D34578





More information about the cfe-commits mailing list