[PATCH] D135251: [LoongArch] Don't merge FrameIndex accesses into [F]{LD,ST}X

[WÁNG Xuěruì via Phabricator via llvm-commits]

xen0n created this revision.
xen0n added reviewers: SixWeining, wangleiat, xry111, MaskRay, gonglingqin.
Herald added subscribers: StephenFan, arphaman, hiraditya.
Herald added a project: All.
xen0n requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Otherwise eliminateFrameIndex cannot figure out how to fixup the stack
offset with its stateless logic, because there wouldn't be an immediate
slot for it to trivially write to, and it may not be easy to transform
the surrounding code to make it work.

This might not be the optimal fix but it fixes a fairly common crash
when compiling moderately complex code with Clang.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D135251

Files:
  llvm/lib/Target/LoongArch/LoongArchISelDAGToDAG.cpp
  llvm/lib/Target/LoongArch/LoongArchISelDAGToDAG.h
  llvm/lib/Target/LoongArch/LoongArchInstrInfo.td
  llvm/lib/Target/LoongArch/LoongArchRegisterInfo.cpp
  llvm/test/CodeGen/LoongArch/ldx-stx-sp-1.ll
  llvm/test/CodeGen/LoongArch/ldx-stx-sp-2.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D135251.465327.patch
Type: text/x-patch
Size: 5115 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221005/82e1004b/attachment.bin>