Boshen
Boshen
@ottomated feel free to work on 4 and 5, I can take a look at 2 later.
I noticed that we still have `tsify`, we can try and get rid of it.
This adds an additional AST pass, but how come it's not shown on the benchmark 🤔 ?
> * move this directly into the mangler Yeah this seems better. We can also add a flag (turned off by default) to enable this.
> Need to slow it down somehow so it matches system allocator better. Probably doesn't matter, we are interested in relative change, not the absolute value from codspeed.
Merge or close?
https://babeljs.io/docs/babel-preset-react#throwifnamespace it's enabled by default in babel.
Better error message. Rolldown should never output `UNHANDLEABLE_ERROR` like this. `throwIfNamespace` needs to be configurable as well.
@Sysix can you help and get this merged.
Do we have a consensus on this PR?