I see, thanks Jeff, just rerun it locally without new failures, will commit It 
soon.

Pan

-----Original Message-----
From: Jeff Law <jeffreya...@gmail.com> 
Sent: Thursday, July 10, 2025 9:22 AM
To: Li, Pan2 <pan2...@intel.com>; gcc-patches@gcc.gnu.org
Cc: juzhe.zh...@rivai.ai; kito.ch...@gmail.com; rdapp....@gmail.com; Chen, Ken 
<ken.c...@intel.com>; Liu, Hongtao <hongtao....@intel.com>
Subject: Re: [PATCH v1] RISCV: Remove the v extension requirement for sat 
scalar run test



On 7/9/25 7:19 PM, Li, Pan2 wrote:
>>>     * gcc.target/riscv/sat/sat_u_trunc-run-6-u8.c: Ditto.
>> Spot checked. Note this uses "RISC-V" so it's not going to be picked up
>> in the patchwork meeting.
> 
> Oops, should be a typo here.
> 
>> It does seem to be causing some testing problems:
> 
> It seems the CI does pick up the latest code, I will send the v2 with RISC-V 
> prefix and see.
> 
> /home/ewlu/precommit-06/_work/gcc-precommit-ci/gcc-precommit-ci/riscv-gnu-toolchain/gcc/gcc/testsuite/gcc.target/riscv/sat/sat_arith.h:7:9:
>  error: unknown type name '__uint128_t'; did you mean '__int128__'?
> compiler exited with status 1
> FAIL: gcc.target/riscv/sat/sat_u_sub-run-4-u32.c -O3 (test for excess errors)
Ah, that means the baseline is stuck.   A problem for Edwin.

With that in mind, I think this is OK.  Go ahead and push it.

jeff

Reply via email to