[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