[PATCH] D151276: Weaken MFI Max Call Frame Size Assertion
Matthias Braun via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 30 11:05:47 PDT 2023
MatzeB added a comment.
> In this case AdjustsStack on line 352 should be initialized to false and MFI.adjustsStack() == AdjustsStack should also be removed making this assertion kind of useless.
If @MatzeB is happy with this, I'm too.
Maybe we can handle this in a similar manner. `AdjustsStack==true` should be the more conservative answer and we should be fine going from an initial `true` to `false` after optimizations. So we can weaken this check as well to only assert that we are not going from `false` to `true`...
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D151276/new/
https://reviews.llvm.org/D151276
More information about the llvm-commits
mailing list