> There's still some nagging issues (which I'll describe in a seperate email), > but > I think we can go ahead and merge this one too to close the loop.
Thanks Vineet, but I'd like to wait the ack from Jeff before commit. Pan -----Original Message----- From: Vineet Gupta <vine...@rivosinc.com> Sent: Saturday, May 3, 2025 1:04 AM To: Jeff Law <jeffreya...@gmail.com>; 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> Subject: Re: [PATCH v1][GCC16-Stage-1] RISC-V: Remove unnecessary frm restore volatile define_insn On 4/30/25 20:44, Jeff Law wrote: >>> Sorry this got backed up as I'm working on FRM overhaul - if this is not >>> super >>> urgent can you please wait for a few weeks for my work to be posted. >>> If you prefer this go in still, fine by me as well. >> Sure thing, feel free to ping me if there is something I can help. > I put Pan's patch onto the deferred list. Looks like Kito's confluence hook already fixes my issue (PR/119164) and in general the FRM writes are much fewer. There's still some nagging issues (which I'll describe in a seperate email), but I think we can go ahead and merge this one too to close the loop. I'll post my current ready to go cleanups, the issues I currently see and the future course of action. Thx, -Vineet