Re: [PATCH version 3, rs6000] Add builtins to convert from float/double to int/long using current rounding mode

2017-09-14 Thread Segher Boessenkool
Hi Carl, On Wed, Sep 13, 2017 at 04:29:01PM -0700, Carl Love wrote: > -- add "TARGET_SF_FPR && TARGET_FPRND" to the define_insn "lrintsfsi2" > as mentioned it was missing on the original define_insn for fctiw. I don't think TARGET_FPRND is correct: this instruction is in the original PowerPC

Re: [PATCH version 3, rs6000] Add builtins to convert from float/double to int/long using current rounding mode

2017-09-13 Thread Carl Love
GCC maintainers: The following patch has been updated to address Segher's comments. -- rename define insn "fctiw" to define_insn "lrintsfsi2" to match the fctid implementation -- add "TARGET_SF_FPR && TARGET_FPRND" to the define_insn "lrintsfsi2" as mentioned it was missing on the original