plugin-legacy for IE11 dosen't work

Describe the bug

Hello, I need some help, please. Just read the steps, ↓

you can glone this repository for more detail vite-for-ie11

1. create a project with vanilla

yarn create vite vite-for-ie11 --template vanilla

1. install plugin @vitejs/plugin-legacy

yarn add @vitejs/plugin-legacy -D

1. create config file and set configs

I just follow the tutorial

touch vite.config.js

// vite.config.js

import legacy from "@vitejs/plugin-legacy";

export default {
  plugins: [
    legacy({
      // for ie11
      targets: ["ie >= 11"],
      additionalLegacyPolyfills: ["regenerator-runtime/runtime"],
    }),
  ],
};

1. add the test code

const apis = ["Proxy", "Reflect", "Promise", "Set", "Map"];

document.querySelector("#app").innerHTML = `
  <h1>Hello Vite!</h1>
  <a href="https://vitejs.dev/guide/features.html" target="_blank">Documentation</a>

  <ol>${apis
    .map((v) => `<li>${v}: ${Boolean(window[v]) ? "yes" : "no"}</li>`)
    .join("")}</ol>
`;

Then I run yarn serve, I got a error msg:

Unhandled promise rejection TypeError: Target is not iterable

Reproduction

https://github.com/Darcrandex/vite-for-ie11.git

System Info

System:
    OS: Windows 10 10.0.18362
    CPU: (4) x64 Intel(R) Core(TM) i5-7500 CPU @ 3.40GHz
    Memory: 7.27 GB / 15.46 GB
  Binaries:
    Node: 14.16.1 - D:\nodejs\node.EXE
    Yarn: 1.22.10 - D:\nodejs\yarn.CMD
    npm: 6.14.12 - D:\nodejs\npm.CMD
  Browsers:
    Chrome: 90.0.4430.212
    Edge: Spartan (44.18362.449.0)
    Internet Explorer: 11.0.18362.1
  npmPackages:
    vite: ^2.4.4 => 2.4.4

Used Package Manager

yarn

Logs

No response

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn’t already an issue that reports the same bug to avoid creating a duplicate.
• Make sure this is a Vite issue and not a framework-specific issue. For example, if it’s a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

[AlexandreBonaventure]

PR here: https://github.com/vitejs/vite/pull/4440

[AlexandreBonaventure]

@patak-js well actually this is not really an upstream bug because:

1. SystemJS relies on Promise.all which requires not only a es.promise polyfill but also the es.array.iterator polyfill see https://github.com/systemjs/systemjs/blob/75853dddde25b13244059babc2657a60196c1b13/src/system-core.js#L143

2. babel-preset-env has no way to know what SystemJS requires in terms of polyfill. This documentation https://babeljs.io/docs/en/babel-plugin-syntax-dynamic-import#working-with-webpack-and-babelpreset-env (this is about webpack internal but this fundamentally the same issue)

So I think we should add es.promise + es6.array.iterator at all time in the legacy plugin because systemJS needs it to operate correctly. It is cleary stated here: https://github.com/systemjs/systemjs#ie11-support