oized (insn) < 0)
> return mode;
>
> - if (reg_mentioned_p (gen_rtx_REG (SImode, FRM_REGNUM), PATTERN (insn)))
> + if (reg_mentioned_p (frm_reg, PATTERN (insn)))
> return get_attr_frm_mode (insn);
>else
> return mode;
>
> Pan
>
> -Original Mes
; 0)
> return mode;
>
> - if (reg_mentioned_p (gen_rtx_REG (SImode, FRM_REGNUM), PATTERN (insn)))
> + if (reg_mentioned_p (frm_reg, PATTERN (insn)))
> return get_attr_frm_mode (insn);
>else
> return mode;
>
> Pan
>
> -Original Message-
>
uly 13, 2023 4:42 PM
To: Kito Cheng
Cc: Jeff Law ; gcc-patches@gcc.gnu.org;
juzhe.zh...@rivai.ai; rdapp@gmail.com; Wang, Yanzhang
Subject: RE: [PATCH v2] RISC-V: Refactor riscv mode after for VXRM and FRM
Sure thing, get you point now, will have a try and send v4 if everything goes
well
: Re: [PATCH v2] RISC-V: Refactor riscv mode after for VXRM and FRM
oh, I know why you failed on that, you need to put it within the
function, not global static, function static variable will construct
when first invoked rather than construct at program start.
Could you try to apply my diff in the
me/pli/repos/gcc/111/riscv-gnu-toolchain/build-gcc-newlib-stage1/./gcc/xgcc
> >
> > -B/home/pli/repos/gcc/111/riscv-gnu-toolchain/build-gcc-newlib-stage1/./gcc/
> > -xc -nostdinc /dev/null -S -o /dev/null
> > -fself-test=../.././gcc/gcc/testsuite/selftests
y, July 13, 2023 2:19 PM
To: Li, Pan2
Cc: Jeff Law ; gcc-patches@gcc.gnu.org;
juzhe.zh...@rivai.ai; rdapp@gmail.com; Wang, Yanzhang
Subject: Re: [PATCH v2] RISC-V: Refactor riscv mode after for VXRM and FRM
Hmmm? I didn't get that error on selftest?
my diff with your v2:
$ git
11:31 PM
> To: Li, Pan2 ; gcc-patches@gcc.gnu.org
> Cc: juzhe.zh...@rivai.ai; rdapp....@gmail.com; Wang, Yanzhang
> ; kito.ch...@gmail.com
> Subject: Re: [PATCH v2] RISC-V: Refactor riscv mode after for VXRM and FRM
>
>
>
> On 7/11/23 23:50, pan2...@intel.com wrote:
> >
.gnu.org
Cc: juzhe.zh...@rivai.ai; rdapp@gmail.com; Wang, Yanzhang
; kito.ch...@gmail.com
Subject: Re: [PATCH v2] RISC-V: Refactor riscv mode after for VXRM and FRM
On 7/11/23 23:50, pan2...@intel.com wrote:
> From: Pan Li
>
> When investigate the FRM dynmaic rounding mode, we find the gl
On 7/11/23 23:50, pan2...@intel.com wrote:
From: Pan Li
When investigate the FRM dynmaic rounding mode, we find the global
unknown status is quite different between the fixed-point and
floating-point. Thus, we separate the unknown function with extracting
some inner common functions.
We wil
;
> Li, Pan2 ; Wang, Yanzhang ;
> kito.cheng
> Subject: Re: [PATCH v2] RISC-V: Refactor riscv mode after for VXRM and FRM
>
>
> +regnum_definition_p (rtx_insn *insn, unsigned int regno)
>
> I prefer it to be reg_set_p.
>
>
>
> +insn_asm_p (rtx_insn *insn)
>
Thank Juzhe for review. Sure, let me hold the v3 for kito's comments.
Pan
From: juzhe.zh...@rivai.ai
Sent: Wednesday, July 12, 2023 2:11 PM
To: Li, Pan2 ; gcc-patches
Cc: Robin Dapp ; jeffreyalaw ; Li,
Pan2 ; Wang, Yanzhang ; kito.cheng
Subject: Re: [PATCH v2] RISC-V: Refactor riscv
+regnum_definition_p (rtx_insn *insn, unsigned int regno)I prefer it to be
reg_set_p.
+insn_asm_p (rtx_insn *insn)asm_insn_p
+global_vxrm_state_unknown_pvxrm_unknown_p
+global_frm_state_unknown_p (rtx_insn *insn)FRM of CALL function is not
"UNKNOWN" unlike VXRM.It just change into another unknown
12 matches
Mail list logo
