On 12/7/23 09:59, Christoph Müllner wrote:
On Thu, Dec 7, 2023 at 11:18 AM Liao Shihua <shi...@iscas.ac.cn> wrote:

In accordance with the suggestions of Christoph Müllner, the following 
amendments are made

Update v1 -> v2:
   1. Rename *_intrinsic-* to *_intrinsic-XLEN.
   2. Typo fix.
   3. Intrinsics with immediate arguments will use marcos at O0 .

It's a little patch add just provides a mapping from the RV intrinsics to the 
builtin
names within GCC.

Thanks for the update!

I think this patchset was not properly tested as I see the tests failing.
Thanks for pointing this out.

I think we're well past the point where if a patchkit doesn't specify how it was tested that it should be rejected. This is standard procedure for the rest of the compiler and there's no good reason why we should have a lax policy in the RISC-V target files.

Jeff

Reply via email to