Incorrect inlining of function with destructuring in initializer of for..of loop

[ajafff]

Bug report or Feature request?

Bug

Version (complete output of terser -V or specific git commit)

What the online REPL currently uses

Complete CLI command or minify() options used

REPL defaults:

{
  module: true,
  compress: {},
  mangle: {},
  output: {},
  parse: {},
  rename: {},
}

terser input

export function caller(data, key) {
    return callee(data, key);
}

function callee(arr, key) {
    for (const { [key]: v } of arr) {
        return v;
    }
}

terser output or error

export function caller(n,r){return function(n,r){for(const{[r]:o}of n)return o}(n)}

Expected result

It should either pass r as second argument to the IIFE or it should not declare r as parameter of the inner function so that it references the variable in the outer scope. Currently r is always undefined at the destructuring

// either
export function caller(n,r){return function(n,r){for(const{[r]:o}of n)return o}(n, r)}

// or
export function caller(n,r){return function(){for(const{[r]:o}of n)return o}()}

Note that this only seems to happen with object destructuring in for..of loops

---

There's a similar case where the reference to the default value of a destructured property is not correctly tracked.

Input:

export function caller(data, fallback) {
    return callee(data, fallback);
}

function callee(arr, fallback) {
    for (const { v = fallback} of arr) {
        return v;
    }
}

Output:

export function caller(n,r){return function(n,r){for(const{v:o=r}of n)return o}(n)}

Same issue: r is always undefined