On Sat, Mar 08, 2025 at 01:52:06PM +0100, Thomas Koenig wrote:
> 
> the attached patch fixes an ICE regresseion where undo state was not
> handled properly when generating formal from actual arguments, which
> occurred under certain conditions with the newly introduced
> -Wexternal-argument-mismatch option.
> 
> The fix is simple: When we are generating these symbols, we no
> longer need to undo anything, so we can just remove them.
> I had considered adding an extra optional argument, but decided
> against it on code clarity grounds.
> 
> While looking at the code, I also saw that a member of gfc_symbol
> introduced with my patch should be a bitfield of width 1.
> 
> OK for trunk?

OK (assuming a successful regression test run).


-- 
Steve

Reply via email to