[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