Only check non-parent monomorphization args

FuelLabs / sway

🌴 Empowering everyone to build reliable and efficient smart contracts.

https://docs.fuel.network/docs/sway/

Apache License 2.0

62.76k stars 5.36k forks source link

Closed xunilrj closed 1 week ago

xunilrj commented 2 weeks ago

The performance improvement is probably coming from just aggregating spans when there is an error.

[x] I have linked to any relevant issues.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have updated the documentation where relevant (API docs, the reference, and the Sway book).
- [ ] If my change requires substantial documentation changes, I have requested support from the DevRel team
[x] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
[x] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[x] I have requested a review from the relevant team or maintainers.

codspeed-hq[bot] commented 2 weeks ago

_{Comparing xunilrj/check-monomorphization-non-parent-only (0705c89) with master (520bfe9)}

⚡ 1 improvements ✅ 21 untouched benchmarks

	Benchmark	`master`	`xunilrj/check-monomorphization-non-parent-only`	Change
⚡	`inlay_hints`	3.8 ms	3.2 ms	+18.83%