Hi! On Thu, Aug 26, 2021 at 09:19:30AM +0800, HAO CHEN GUI wrote: > gcc/ > ?????? * config/rs6000/rs6000-call.c (rs6000_gimple_fold_builtin): > ?????? Modify the VSX_BUILTIN_XVMINDP, ALTIVEC_BUILTIN_VMINFP, > ?????? VSX_BUILTIN_XVMAXDP, ALTIVEC_BUILTIN_VMAXFP expansions.
Something mangles your email, giving all those question marks. It is hard to review like this. Don't send patches as replies in threads please. It is impossible to keep track of such things. > --- a/gcc/config/rs6000/rs6000-call.c > +++ b/gcc/config/rs6000/rs6000-call.c > @@ -12159,6 +12159,11 @@ rs6000_gimple_fold_builtin > (gimple_stmt_iterator *gsi) > ???????????? return true; > ???????? /* flavors of vec_min.?? */ > ???????? case VSX_BUILTIN_XVMINDP: > +?????? case ALTIVEC_BUILTIN_VMINFP: > +?????????? if (!flag_finite_math_only || flag_signed_zeros) > +?????? return false; > +?????????? /* Fall through to MIN_EXPR.?? */ > +?????????? gcc_fallthrough (); > ???????? case P8V_BUILTIN_VMINSD: > ???????? case P8V_BUILTIN_VMINUD: > ???????? case ALTIVEC_BUILTIN_VMINSB: Yeah I would rather not review this like this :-) Segher
