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

[mono] Don't use recursion in mono_ssa_rename_vars #61677

Merged

Commits on Nov 16, 2021

  1. [mono] Don't use recursion in mono_ssa_rename_vars

    Implements dotnet#60266
    
    Avoid deep recursion in mono_ssa_rename_vars, change the way we traverse
    dominated bb's. Instead of using recursion, use stack like array to
    store information about stack history and the traversal.
    
    The performance remains the same (or is slightly better) than before.
    Times for the compilation of repro from dotnet#57141
    (JIT time minimum from 5 runs):
    
    Before:
    
        LLVM output file: './Uno.UI.FluentTheme.dll.bc.tmp'.
        JIT time: 4810 ms, Generation time: 2040 ms, Assembly+Link time: 2 ms.
    
    After:
    
        LLVM output file: './Uno.UI.FluentTheme.dll.bc.tmp'.
        JIT time: 4781 ms, Generation time: 2017 ms, Assembly+Link time: 2 ms.
    radekdoulik committed Nov 16, 2021
    Configuration menu
    Copy the full SHA
    b747f3b View commit details
    Browse the repository at this point in the history
  2. Fix assignment

    radekdoulik committed Nov 16, 2021
    Configuration menu
    Copy the full SHA
    7064afb View commit details
    Browse the repository at this point in the history

Commits on Nov 23, 2021

  1. Apply suggestions from code review

    Co-authored-by: imhameed <imhameed@microsoft.com>
    radekdoulik and imhameed authored Nov 23, 2021
    Configuration menu
    Copy the full SHA
    f9bd589 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    701c70e View commit details
    Browse the repository at this point in the history