On Mon, 10 May 2021 at 23:49, David Rowley <dgrowle...@gmail.com> wrote: > > On Mon, 10 May 2021 at 21:16, Greg Nancarrow <gregn4...@gmail.com> wrote: > > During debugging I noticed some code in ExecResult() where a WHILE > > loop is being used with an unconditional RETURN at the end of the > > block (which is intentional, looking at the history of changes), but > > now there's no actual use of the loop in any way. The code should > > probably be changed to just use IF for clarity. > > I've attached a patch. > > Looks like leftovers from ea15e1867. > > I don't think this will affect any code generation but you are right, > it should be an "if".
Since there's no bug fix here, I thought that there's not much point in backpatching this. Does anyone object to making this small change in master? David
