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.

Sign up for GitHub

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

Jump to bottom

[SystemZ] Assign the full space for promoted and split outgoing args. #94

Merged
merged 1 commit into from
Mar 9, 2021
Merged

[SystemZ] Assign the full space for promoted and split outgoing args. #94

merged 1 commit into from
Mar 9, 2021

Commits on Mar 9, 2021

  1. [SystemZ] Assign the full space for promoted and split outgoing args. 

    When a large "irregular" (e.g. i96) integer call argument is converted to
indirect, 64-bit parts are stored to the stack. The full stack space
(e.g. i128) was not allocated prior to this patch, but rather just the exact
space of the original type. This caused neighboring values on the stack to be
overwritten.

Thanks to Josh Stone for reporting this.

Review: Ulrich Weigand
Fixes https://bugs.llvm.org/show_bug.cgi?id=49322
Differential Revision: https://reviews.llvm.org/D97514

(cherry picked from commit 52bbbf4)
    @JonPsson @cuviper
    JonPsson authored and cuviper committed Mar 9, 2021
    Configuration menu
    Copy the full SHA
    19d7afd View commit details
    Browse the repository at this point in the history