Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recover AST of a declarations file #817

Closed
JohnnyMorganz opened this issue Jan 24, 2023 · 1 comment · Fixed by #860
Closed

Recover AST of a declarations file #817

JohnnyMorganz opened this issue Jan 24, 2023 · 1 comment · Fixed by #860
Assignees
Labels
enhancement New feature or request

Comments

@JohnnyMorganz
Copy link
Contributor

I want to be able to support documentation comments inlined within a declarations file. For this, I need the AST of the file to be able to find the comments located within it, and attach them to the appropriate functions etc.

Currently, the SourceModule (which contains the AST) created for a declarations file is discarded after its used

@vegorov-rbx
Copy link
Collaborator

Should be available in the next sync.

andyfriesen added a commit that referenced this issue Mar 10, 2023
* Fix #817 
* Fix #850 
* Optimize math.floor/ceil/round with SSE4.1
    * Results in a ~7-9% speedup on the math-cordic benchmark.
* Optimized table.sort.
* table.sort is now ~4.1x faster (when not using a predicate) and ~2.1x
faster when using a simple predicate. Performance may improve further in
the future.
* Reorganize the memory ownership of builtin type definitions.
* This is a small initial step toward affording parallel typechecking.

The new type solver is coming along nicely. We are working on fixing
crashes and bugs.

A few major changes to native codegen landed this week:
* Fixed lowering of Luau IR mod instruction when first argument is a
constant
* Added VM register data-flow/capture analysis
* Fixed issues with optimizations in unreachable blocks

---------

Co-authored-by: Arseny Kapoulkine <[email protected]>
Co-authored-by: Vyacheslav Egorov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

Successfully merging a pull request may close this issue.

2 participants