[RESEND PATCH v5] RISC-V: use fclass insns to implement isfinite, isnormal and isinf builtins

2024-08-15 Thread Vineet Gupta
This was approved but resending so CI can retest after the ranges fixes. --- Currently these builtins use float compare instructions which require FP flags to be saved/restored which could be costly in uarch. RV Base ISA already has FCLASS.{d,s,h} instruction to compare/identify FP values w/o dist

Re: [PATCH v5] RISC-V: use fclass insns to implement isfinite,isnormal and isinf builtins

2024-07-17 Thread Jeff Law
On 7/16/24 12:26 PM, Vineet Gupta wrote: Hi Jeff, On 7/13/24 07:54, Vineet Gupta wrote: Changes since v4: - No functional changes. - Use int iterator and attr to implement expanders in md (inspired by loongarch patch. Thx Xi Ruoyao) Changes since v3: - Remove '*' from define_i

Re: [PATCH v5] RISC-V: use fclass insns to implement isfinite,isnormal and isinf builtins

2024-07-16 Thread Vineet Gupta
Hi Jeff, On 7/13/24 07:54, Vineet Gupta wrote: > Changes since v4: > - No functional changes. > - Use int iterator and attr to implement expanders in md > (inspired by loongarch patch. Thx Xi Ruoyao) > > Changes since v3: > - Remove '*' from define_insn for fclass > - Remove the dummy

[PATCH v5] RISC-V: use fclass insns to implement isfinite, isnormal and isinf builtins

2024-07-13 Thread Vineet Gupta
Changes since v4: - No functional changes. - Use int iterator and attr to implement expanders in md (inspired by loongarch patch. Thx Xi Ruoyao) Changes since v3: - Remove '*' from define_insn for fclass - Remove the dummy expander for fclass. - De-duplicate the expanders code by usi