Yes, thanks Robin, we can move to another mail thread for this, avoiding misleading.
Pan -----Original Message----- From: Robin Dapp <[email protected]> Sent: Tuesday, July 4, 2023 9:15 PM To: Li, Pan2 <[email protected]>; [email protected]; gcc-patches <[email protected]> Cc: [email protected]; jeffreyalaw <[email protected]>; Wang, Yanzhang <[email protected]>; kito.cheng <[email protected]> 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
