[PATCH] D42600: [CodeGen][Shrink-wrap]split restore point

Sushant Gokhale via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 9 06:39:52 PDT 2023


sushgokh added a comment.

In D42600#4327559 <https://reviews.llvm.org/D42600#4327559>, @efriedma wrote:

> It looks like for 32-bit ARM, shrink-wrapping is blocking the formation of `pop {pc}` instructions to return from a function (so we instead `pop {lr}; bx lr`).  Is this new, or a side-effect of shrink-wrapping more cases?  Any idea what's causing it?  Is there an issue tracking it?

This should be side effect of doing more shrink wrapping and caused by later passes as this patch just changes the save/restore point and nothing beyond that. Currently, we dont have any issue tracking it but if this is leading to performance regression, then could you please raise a bug for it  when I reland it ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D42600



More information about the llvm-commits mailing list