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

Allow the inliner to substitute for small arguments #69068

Merged
merged 4 commits into from
May 17, 2022

Commits on May 9, 2022

  1. Allow the inliner to substitute for small arguments

    Since we no longer have contextual nodes to indicate small arguments,
    this should be safe to do. For register arguments it does not matter and
    for stack arguments it is handled in codegen (for macOS arm64).
    jakobbotsch committed May 9, 2022
    Configuration menu
    Copy the full SHA
    e040c32 View commit details
    Browse the repository at this point in the history
  2. More work to enable this

    With small args we could end up inserting a cast between the arg node
    and a GT_RET_EXPR. We would not handle substituting this GT_RET_EXPR
    correctly. This reverts some more of the changes from 3e65d68 to make
    the scenario work again, by skipping over GT_RET_EXPR before creating
    the inlinee arg nodes.
    jakobbotsch committed May 9, 2022
    Configuration menu
    Copy the full SHA
    154cf95 View commit details
    Browse the repository at this point in the history

Commits on May 13, 2022

  1. Configuration menu
    Copy the full SHA
    3d0cdb2 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    e64a5dc View commit details
    Browse the repository at this point in the history