Switch sources to typed arrays instead of strings

[cd1m0]

Warning

This is an interface breaking change.

Change our implementation to keep source file contents as Uint8Arrays instead of string. This is needed since AST source map are actually byte offsets into the file. We were erroneously trearting them as string indices, which breaks in the presence of multi-byte Unicode characters.

This involves:

1. Changing a bunch of the compile* interfaces to accept Map<string, Uint8Array> instead of Map<string, string>.
2. Changing the StructuredDocumentation pass to correctly compute docs by doing byte indexing.
3. Changing the ASTWriter to compute a correct map.
4. Adding tests.

Todo (@blitz-1306 ): I havent looked at inline assembly. Is there anything to do there?

[blitz-1306]

Great work. Two things to consider:

1. There is a coverage failure.
2. One small nit in the code.

I will try to pick up issues tomorrow.

I havent looked at inline assembly. Is there anything to do there?

No, I think there is nothing left.

[blitz-1306]

Looks ready to merge after 73492b7.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9ebcad8) 91.59% compared to head (685b922) 91.60%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #242   +/-   ##
=======================================
  Coverage   91.59%   91.60%           
=======================================
  Files         267      270    +3     
  Lines        6630     6634    +4     
  Branches     1353     1350    -3     
=======================================
+ Hits         6073     6077    +4     
  Misses        291      291           
  Partials      266      266           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.