[PATCH] D39611: [CodeGen] change const-ness of complex calls

Sanjay Patel via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 10 09:16:18 PST 2017


spatel added a comment.

Thanks for the clarification!

If I'm reading this properly, we should make the same kind of change as in https://reviews.llvm.org/D39481 ('c' -> 'e') for most of complex.h. Ie, the standard allows errno-setting, and it's (unfortunately for optimization) even more clearly stated in the newer additions to the standards.

We can leave these functions as always constant ('c') because they don't actually do any math and therefore won't set errno:
cimag ( http://en.cppreference.com/w/c/numeric/complex/cimag )
creal ( http://en.cppreference.com/w/c/numeric/complex/creal )
cproj ( http://en.cppreference.com/w/c/numeric/complex/cproj )
conj (http://en.cppreference.com/w/c/numeric/complex/conj )


https://reviews.llvm.org/D39611





More information about the cfe-commits mailing list