On 11/7/22 15:07, Palmer Dabbelt wrote:
On Thu, 03 Nov 2022 15:23:28 PDT (-0700), j...@ventanamicro.com wrote:
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.
Which one (or either)? I'm fine with either option, just trying to
avoid another re-spin as this one is a bit vague.
Sorry I wasn't clear. Either is fine with me.
Jeff