search for: r66652

...ith the original source code, where everything works fine, besides the optimized version of sqrt(NA) giving NaN. Did something perhaps change in gcc 5 for the ISNAN macro that makes it unsuitable to be used in this way? FWIW it looks like the original fix was introduced by Luke about a year ago in r66652: https://github.com/wch/r-source/commit/e9aaf8fdeddf27c2a9078cd214a41475c8ff6f40

On 14/09/2015 9:36 AM, luke-tierney at uiowa.edu wrote: > I believe the issue is that on Windows the sqrt function when called > with a NaN does not return the same NaN, as it does on other platforms. > We have > > #if (defined(_WIN32) || defined(_WIN64)) && defined(__GNUC__) && \ > __GNUC__ <= 4 > # define R_sqrt(x) (ISNAN(x) ? x : sqrt(x)) > #else