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". David