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

[AMDGPU] Include WWM register spill into BB Prolog #111496

Merged
merged 1 commit into from
Oct 8, 2024

Commits on Oct 8, 2024

  1. [AMDGPU] Include WWM register spill into BB Prolog

    With llvm#93526 we split the regalloc pipeline further
    to have a standalone allocation for wwm registers
    and per-lane VGPRs. Currently the presence of the
    wwm-spill reloads inserted at the bb-top limits the
    isBasicPrologue function during the per-lane vgpr
    regalloc to skip past the exec manipulation instruction
    and ended up causing incorrect codegen. The wmm-spill
    inserted during the wwm-regalloc pipeline should also
    be included in the bb-prolog so that the per-lane vgpr
    regalloc pipeline can identify the appropriate insertion
    points for their spills and copies.
    cdevadas committed Oct 8, 2024
    Configuration menu
    Copy the full SHA
    c902221 View commit details
    Browse the repository at this point in the history