nodejs / performance

Node.js team focusing on performance
MIT License
377 stars 7 forks source link

Revisit Fast APIs implementations & New implementations #182

Open H4ad opened 2 weeks ago

H4ad commented 2 weeks ago

Since #23, there have been many implementations in Node using fast APIs, and I want to open this issue to encourage to revisit them and probably take a look at other places where we could implement the latest changes.

Since https://github.com/nodejs/node/pull/54077, we have now support to throw errors inside fast apis, this was not possible before and we the "options.fallback" was the only way to throw errors.

We also had some limitations around int64 and bigints, and now I think those limitations was solved since I found some tests on v8 returning bigints:

I'm still trying to figure out all the limitations around these new changes, so any hints around these new changes are very welcome.

billywhizz commented 2 weeks ago

i'm keen to help out with this effort @H4ad. i've done quite a lot with fast api on my own runtime and will be contributing to node.js as much as I can in period ahead. will take a look over next week or so and will add anything i think would be useful to this issue.