{"push_id":1273745786,"size":3,"distinct_size":3,"ref":"refs/heads/master","head":"9d84d0e51d0a590024a050b64e04df3214a04a01","before":"feb62ddacb7b1d772d7383de0228a3977f07fc1e","commits":[{"sha":"32efd690bd9ab8cb55d185bf5b0a1a3c0e539120","author":{"name":"Siddhesh Poyarekar","email":"44e7aba8836efc166542081a0adb36c5bf9396c9@sourceware.org"},"message":"Consolidate reduce_and_compute code\n\nThis patch reshuffles the reduce_and_compute code so that the\nstructure matches other code structures of the same type elsewhere in\ns_sin.c and s_sincos.c.  This is the beginning of an attempt to\nconsolidate and reduce code duplication in functions in s_sin.c to\nmake it easier to read and possibly also easier for the compiler to\noptimize.\n\n\t* sysdeps/ieee754/dbl-64/s_sin.c (reduce_and_compute):\n\tConsolidate switch cases 0 and 2.","distinct":true,"url":"https://api.github.com/repos/francois-wellenreiter/glibc/commits/32efd690bd9ab8cb55d185bf5b0a1a3c0e539120"},{"sha":"1a822c61844eb378bc8d676f26edf1a0285303b1","author":{"name":"Siddhesh Poyarekar","email":"44e7aba8836efc166542081a0adb36c5bf9396c9@sourceware.org"},"message":"Add fall through comments\n\nAdd fall through comments I had missed writing in previously.","distinct":true,"url":"https://api.github.com/repos/francois-wellenreiter/glibc/commits/1a822c61844eb378bc8d676f26edf1a0285303b1"},{"sha":"9d84d0e51d0a590024a050b64e04df3214a04a01","author":{"name":"Siddhesh Poyarekar","email":"44e7aba8836efc166542081a0adb36c5bf9396c9@sourceware.org"},"message":"Use fabs(x) instead of branching on signedness of input to sin and cos\n\nThe sin and cos code is inconsistent about its use of fabs to get the\nabsolute value of X where in some places it conditionalizes the code\nwhile in others it uses fabs.  fabs seems to be a better candidate in\nmost cases because it avoids a branch.  Similarly there is an attempt\nto make it easier for the compiler to emit conditional assignment\ninstructions (like fcsel on aarch64) where it can, by isolating\nconditional assignment constructs from the rest of the expression.\n\nA further benefit of this change is to identify common constructs\nacross functions and consolidate them in future patches.\n\n\t* sysdeps/ieee754/dbl-64/s_sin.c (do_cos_slow): Use ternary\n\tinstead of if/else.\n\t(do_sin_slow): Likewise.\n\t(do_sincos_1): Use fabs instead of if/else.\n\t(do_sincos_2): Likewise.\n\t(__sin): Likewise.\n\t(__cos): Likewise.\n\t(slow2): Likewise.\n\t(sloww): Likewise.\n\t(sloww1): Likewise.  Drop argument M.\n\t(sloww2): Use fabs instead of if/else.\n\t(bsloww): Likewise.\n\t(bsloww1): Likewise.\n\t(bsloww2): Likewise.","distinct":true,"url":"https://api.github.com/repos/francois-wellenreiter/glibc/commits/9d84d0e51d0a590024a050b64e04df3214a04a01"}]}
{"actor":{"display_login":"francois-wellenreiter"}}