[all-commits] [llvm/llvm-project] 69bec0: [X86] Correct 32-bit immediate assertion and fix 6...

[Wesley Wiser via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 69bec0afbb8f2aa0021d18ea38768360b16583a9
      https://github.com/llvm/llvm-project/commit/69bec0afbb8f2aa0021d18ea38768360b16583a9
  Author: Wesley Wiser <wwiser at gmail.com>
  Date:   2025-08-03 (Sun, 03 Aug 2025)

  Changed paths:
    M llvm/lib/CodeGen/PrologEpilogInserter.cpp
    M llvm/lib/Target/X86/X86FrameLowering.cpp
    M llvm/lib/Target/X86/X86RegisterInfo.cpp
    M llvm/lib/Target/X86/X86RegisterInfo.h
    A llvm/test/CodeGen/X86/avx512f-large-stack.ll
    M llvm/test/CodeGen/X86/huge-stack.ll
    A llvm/test/CodeGen/X86/large-displacements-fastisel.ll
    A llvm/test/CodeGen/X86/large-displacements.ll
    M llvm/test/CodeGen/X86/merge-huge-sp-updates.ll
    M llvm/test/CodeGen/X86/stack-clash-extra-huge.ll
    M llvm/test/CodeGen/X86/stack-clash-huge.ll
    M llvm/test/CodeGen/X86/win64-stackprobe-overflow.ll

  Log Message:
  -----------
  [X86] Correct 32-bit immediate assertion and fix 64-bit lowering for huge frame offsets (#123872)

The assertion previously did not work correctly because the operand was
being truncated to an `int` prior to comparison.

Change the assertion into a a reported error as suggested in
https://github.com/llvm/llvm-project/pull/101840#issuecomment-2304992425
by @arsenm

Finally, fix the lowering on 64-bit targets so that offsets larger than
32-bit are correctly addressed and add tests for various reported
issues.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications