Yes, thanks Robin, we can move to another mail thread for this, avoiding misleading.
Pan -----Original Message----- From: Robin Dapp <rdapp....@gmail.com> Sent: Tuesday, July 4, 2023 9:15 PM To: Li, Pan2 <pan2...@intel.com>; juzhe.zh...@rivai.ai; gcc-patches <gcc-patches@gcc.gnu.org> Cc: rdapp....@gmail.com; jeffreyalaw <jeffreya...@gmail.com>; Wang, Yanzhang <yanzhang.w...@intel.com>; kito.cheng <kito.ch...@gmail.com> Subject: Re: [PATCH v1] RISC-V: Fix one typo of FRM dynamic definition > Just revert this patch, it reports some weird illegal instr, I may > need more time for this. The illegal instruction is due to the wrong rounding mode. We set 5 instead of 7 because the two enums don't match. A simple but ugly fix would be two dummy entries so that FRM_MODE_DYN is entry 7 in enum attr_frm_mode but I'd prefer not to do that :) Regards Robin