fix(semantic): ModuleRecord's indirect_export_entires missing reexported imports

[Dunqing]

local_name has name and span. We can't directly compare ExportEntry's local_name and ImportEntry's local_name. because the span is never equal.

[Dunqing]

• fix(semantic): incorrect ExportEntry span for ExportAllDeclaration in ModuleRecord #2793
• fix(semantic): ModuleRecord's indirect_export_entires missing reexported imports #2792 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Dunqing and the rest of your teammates on Graphite

[codspeed-hq]

CodSpeed Performance Report

Merging #2792 will improve performances by 13.63%

_{Comparing 03-23-fix_semantic_ModuleRecord_s_indirect_export_entires_missing_reexported_imports (93e7b42) with main (4a42c5f)}

Summary

⚡ 1 improvements
✅ 33 untouched benchmarks

Benchmarks breakdown

	Benchmark	main	03-23-fix_semantic_ModuleRecord_s_indirect_export_entires_missing_reexported_imports	Change
⚡	semantic[cal.com.tsx]	257.9 ms	226.9 ms	+13.63%

[Boshen]

Can you double check with the spec? I want to make sure that we are on the same page.

https://tc39.es/ecma262/#sec-parsemodule step 10

[Dunqing]

Can you double check with the spec? I want to make sure that we are on the same page.

https://tc39.es/ecma262/#sec-parsemodule step 10

I have been checked, comments and behaviour consistent with the spec