[llvm] [LLVM] [X86] Fix integer overflows in frame layout for huge frames (PR #101840)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 22 08:22:53 PDT 2024


arsenm wrote:

> I believe that's the same invocation I looked at to the get the assembly I posted above. If I'm understanding everything correctly, currently that code is miscompiled on 32-bit x86 and after my change (for clang/LLVM builds with assertions enabled) it will now instead trigger the assertion.

But it does mean it should be upgraded to a proper backend error, instead of an assertion or verifier error. Preferably using DiagnosticInfo 

https://github.com/llvm/llvm-project/pull/101840


More information about the llvm-commits mailing list