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

Share return value handling in trim analysis #101398

Merged
merged 7 commits into from
Apr 25, 2024
Merged

Commits on Apr 22, 2024

  1. Configuration menu
    Copy the full SHA
    2b64adc View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    56d1e24 View commit details
    Browse the repository at this point in the history

Commits on Apr 24, 2024

  1. Configuration menu
    Copy the full SHA
    a3e3c88 View commit details
    Browse the repository at this point in the history
  2. Avoid throwing from analyzer in Release builds

    And add missing cases to the switch statement for the analyzer.
    sbomer committed Apr 24, 2024
    Configuration menu
    Copy the full SHA
    3b8cc81 View commit details
    Browse the repository at this point in the history
  3. Fix ILC callsites

    sbomer committed Apr 24, 2024
    Configuration menu
    Copy the full SHA
    69017ea View commit details
    Browse the repository at this point in the history
  4. Fix Array_CreateInstance case

    The PInvoke logic and the CheckAndReportRequires logic
    should not be needed for this intrinsic, and the old shared
    HandleCallAction would go to the default case that sets the
    return value to Top and returns false.
    
    Now this instead returns true and lets the shared return
    value logic kick in.
    sbomer committed Apr 24, 2024
    Configuration menu
    Copy the full SHA
    b898b27 View commit details
    Browse the repository at this point in the history

Commits on Apr 25, 2024

  1. PR feedback

    - Return instead of out param in HandleCall
    - FIx formatting
    sbomer committed Apr 25, 2024
    Configuration menu
    Copy the full SHA
    c450da6 View commit details
    Browse the repository at this point in the history