Skip to content

partially revert 6f6cb1714365a #1918

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
Closed
nickdesaulniers opened this issue Aug 17, 2023 · 1 comment
Closed

partially revert 6f6cb1714365a #1918

nickdesaulniers opened this issue Aug 17, 2023 · 1 comment
Labels
duplicate This issue or pull request already exists

Comments

@nickdesaulniers
Copy link
Member

nickdesaulniers commented Aug 17, 2023 •
edited
Loading

6f6cb17 mentions 5k+ stack usage. (#1455)

After clang-18's (https://reviews.llvm.org/rGe698695fbbf62e6676f8907665187f2d2c4d814b), I only see:

$ ARCH=arm make LLVM=1 -j128 allyesconfig drivers/gpu/drm/
drivers/gpu/drm/amd/amdgpu/../display/dc/dml/calcs/dce_calcs.c:77:13: error: stack frame size (1056) exceeds limit (1024) in 'calculate_bandwidth' [-Werror,-Wframe-larger-than]
   77 | static void calculate_bandwidth(
      |             ^

Filing an issue about this because I don't think we have one specific to that kernel commit.

Tangentially related to #1752.

See also: llvm/llvm-project#41896
@nickdesaulniers
Copy link
Member Author

dup'ing to #1455

@nickdesaulniers nickdesaulniers added duplicate This issue or pull request already exists and removed [ARCH] arm32 This bug impacts ARCH=arm -Wframe-larger-than= labels Aug 17, 2023
@nickdesaulniers nickdesaulniers mentioned this issue Aug 17, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
duplicate This issue or pull request already exists
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nickdesaulniers