Re: [PATCH] RISC-V missing __builtin_lceil and __builtin_lfloor

2022-11-07 Thread Kevin Lee
> Kevin: Looks like this got corrupted, possibly from copy/paste into > gmail. I resurrect it, but there's a floating-point test failure in > gfortran. Looks like it predates this, but I'm trying to bisect it to > at least have a root cause before just ignoring it. I've got this > floating aroun

Re: [PATCH] RISC-V missing __builtin_lceil and __builtin_lfloor

2022-10-02 Thread Kevin Lee
Thank you for the update Palmer. I'll certainly look into the corrupted patch issue and the floating-point test failure in gfortran. On Sun, Oct 2, 2022 at 1:42 PM Palmer Dabbelt wrote: > On Sat, 17 Sep 2022 14:16:13 PDT (-0700), Kito Cheng wrote: > > LGTM, thanks, I guess I just missed this bef

Re: [PATCH] RISC-V missing __builtin_lceil and __builtin_lfloor

2022-10-02 Thread Palmer Dabbelt
On Sat, 17 Sep 2022 14:16:13 PDT (-0700), Kito Cheng wrote: LGTM, thanks, I guess I just missed this before No worries, I'd just stubmled on it looking through old stuff. Kevin: Looks like this got corrupted, possibly from copy/paste into gmail. I resurrect it, but there's a floating-point t

Re: [PATCH] RISC-V missing __builtin_lceil and __builtin_lfloor

2022-09-17 Thread Kito Cheng via Gcc-patches
LGTM, thanks, I guess I just missed this before Palmer Dabbelt 於 2022年9月17日 週六 23:07 寫道: > On Mon, 15 Aug 2022 17:44:35 PDT (-0700), kev...@rivosinc.com wrote: > > Hello, > > Currently, __builtin_lceil and __builtin_lfloor doesn't generate an > > existing instruction fcvt, but rather calls ceil

Re: [PATCH] RISC-V missing __builtin_lceil and __builtin_lfloor

2022-09-17 Thread Palmer Dabbelt
On Mon, 15 Aug 2022 17:44:35 PDT (-0700), kev...@rivosinc.com wrote: Hello, Currently, __builtin_lceil and __builtin_lfloor doesn't generate an existing instruction fcvt, but rather calls ceil and floor from the library. This patch adds the missing iterator and attributes for lceil and lfloor to

[PATCH] RISC-V missing __builtin_lceil and __builtin_lfloor

2022-08-15 Thread Kevin Lee
Hello, Currently, __builtin_lceil and __builtin_lfloor doesn't generate an existing instruction fcvt, but rather calls ceil and floor from the library. This patch adds the missing iterator and attributes for lceil and lfloor to produce the optimized code. The test cases check the correct generatio