[PATCH] D145471: [RISCV] Set how many bytes load from or store to stack slot

luxufan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 12 03:43:02 PDT 2023


StephenFan added a comment.

This patch caused a code size regression in our ci tests https://github.com/dtcxzyw/llvm-ci/issues/8. And part of the extra codes it generates is as follows:
F26778812: WechatIMG3384.jpeg <https://reviews.llvm.org/F26778812>

I think the root cause is this patch makes the original function

  unsigned isLoadFromStackSlot(const MachineInstr &MI,
                                 int &FrameIndex) const override;

no longer be overridden.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145471



More information about the llvm-commits mailing list