On 11/2/22 18:26, Palmer Dabbelt wrote:

I also tried to remove that restriction but it looks like it can't
work because we can't create
pseudo-registers during shrink wrapping and shrink wrapping can't work either.

I believe this means that shrink wrapping cannot interfere with a long
stack frame
so there is nothing to test against in this case?

It'd be marginally better to have such a test case to ensure we don't
shrink wrap it -- that would ensure that someone doesn't accidentally
introduce shrink wrapping with large offsets.   Just a bit of future
proofing.

If there's passing test cases that fail with that check removed then it's probably good enough, though I think in this case just having a comment there saying why the short-stack check is necessary should be fine.

I can live with this.

jeff

Reply via email to