[PATCH] D70427: [SystemZ] The Local Area Offset in SystemZFrameLowering.

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 21 09:44:52 PST 2019


uweigand added a comment.

Right.  It seems we really do need to implement the change I described in my last comment then.  I've just quickly tried this patch:
F10820411: diff-spilloffsettable <https://reviews.llvm.org/F10820411>
and this seems to fix the problem for your test case:

  anyregcc1:                              # @anyregcc1
          .cfi_startproc
  # %bb.0:                                # %entry
          std     %f0, 128(%r15)          # 8-byte Folded Spill
          .cfi_offset %f0, -32
          #APP
          #NO_APP
          ld      %f0, 128(%r15)          # 8-byte Folded Reload
          br      %r14

Can you check whether this doesn't introduce any regressions anywhere, and integrate your comment and test case?  Thanks!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70427/new/

https://reviews.llvm.org/D70427





More information about the llvm-commits mailing list