Re: Re: [PATCH] RISC-V: Add runtime invariant support

2022-08-23 Thread juzhe.zh...@rivai.ai
I tried #define LOGICAL_OP_NON_SHORT_CIRCUIT 1 in RISC-V port. The warning is still there. Are you considering this patch:https://gcc.gnu.org/pipermail/gcc-patches/2022-August/600120.html to solve this issue ? Or you are trying another solution to fix this ? juzhe.zh...@rivai.ai From: Richa

Re: Re: [PATCH] RISC-V: Add runtime invariant support

2022-08-20 Thread Andrew Pinski via Gcc-patches
to.cheng; andrew; rguenther > Subject: Re: Re: [PATCH] RISC-V: Add runtime invariant support > On Sat, Aug 20, 2022 at 5:06 PM 钟居哲 wrote: > > > > Hi, it seems that this warning still report if I revert my patch. Am I > > right? Feel free to correct me. Maybe I need to try

Re: Re: [PATCH] RISC-V: Add runtime invariant support

2022-08-20 Thread 钟居哲
OK. Thank you. I am gonna try it again and fix this in RISC-V port. juzhe.zh...@rivai.ai From: Andrew Pinski Date: 2022-08-21 08:18 To: 钟居哲 CC: Andreas Schwab; gcc-patches; kito.cheng; andrew; rguenther Subject: Re: Re: [PATCH] RISC-V: Add runtime invariant support On Sat, Aug 20, 2022 at 5

Re: Re: [PATCH] RISC-V: Add runtime invariant support

2022-08-20 Thread Andrew Pinski via Gcc-patches
On Sat, Aug 20, 2022 at 5:06 PM 钟居哲 wrote: > > Hi, it seems that this warning still report if I revert my patch. Am I right? > Feel free to correct me. Maybe I need to try it again? The warning will not be still there. The reason is NUM_POLY_INT_COEFFS defaults to 1 which means vf.is_constant (&

Re: Re: [PATCH] RISC-V: Add runtime invariant support

2022-08-20 Thread 钟居哲
Hi, it seems that this warning still report if I revert my patch. Am I right? Feel free to correct me. Maybe I need to try it again? juzhe.zh...@rivai.ai From: Andrew Pinski Date: 2022-08-21 07:53 To: Andreas Schwab CC: juzhe.zhong; gcc-patches; kito.cheng; andrew; Richard Guenther Subject: R