[PATCH] D59836: PEI: Delay checking requiresFrameIndexReplacementScavenging

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 26 13:28:46 PDT 2019


arsenm added a comment.

In D59836#1443502 <https://reviews.llvm.org/D59836#1443502>, @thegameg wrote:

> It would be good to have a test case for this, but I assume you're planning on changing `SIRegisterInfo::requiresFrameIndexReplacementScavenging`? Maybe you can merge both patches since AMDGPU seems to be the only user of the hook.


I usually try to separate the target specific part when possible: D59840 <https://reviews.llvm.org/D59840>


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

https://reviews.llvm.org/D59836





More information about the llvm-commits mailing list