On Mon, Aug 25, 2025 at 09:36:44AM +1000, Richard Henderson wrote: > On 8/25/25 08:27, Edgar E. Iglesias wrote: > > From: "Edgar E. Iglesias" <edgar.igles...@amd.com> > > > > Break out raise_divzero(). No functional change. > > > > Signed-off-by: Edgar E. Iglesias <edgar.igles...@amd.com> > > --- > > target/microblaze/op_helper.c | 30 ++++++++++++++---------------- > > 1 file changed, 14 insertions(+), 16 deletions(-) > > You could usefully squash this with the previous. It obviates the b -> > divisor rename, and it perhaps clarifies why you don't like using "ra" in > two different contexts.
Yes, that was what prompted me to rename ra in the first place. I'll squash this with the former patch and rename pc to unwind_pc, keeping your RB tag. Let me know if that wasn't your intention. Cheers, Edgar > > Reviewed-by: Richard Henderson <richard.hender...@linaro.org> > > > r~
